Team info
All meeting notes
Attendees: @infinisil @0x4A6F @das_j @tomberek Sergey
Notes
- Project board: Nix formatting · GitHub
- Assign priorities, seems to work well
- Workflow ideas: Have a label for “needs team-review” vs “can be closed by individuals”
- Closed Discover Nix files · Issue #152 · NixOS/nixfmt · GitHub
- Closed Easy GitHub Actions integration · Issue #180 · NixOS/nixfmt · GitHub
- Closed [WIP] Sort assignments in sets and "let" by KAction · Pull Request #61 · NixOS/nixfmt · GitHub
- Merged treewide: Rename nixfmt to nixfmt-classic by dasJ · Pull Request #300468 · NixOS/nixpkgs · GitHub
- Regarding the future of the CLI command
- With a Nix PR to set
nix fmt
to usenixfmt
and introducenix-fmt
for the traditional CLI- Would also be very confusing though
- Problem: If
nix fmt
by default callsnixfmt
, what if somebody wants to change the default to e.g.treefmt
, how doestreefmt
callnixfmt
then?-
nixfmt-official
? If we don’t need a good CLI interface it can be longer
-
- No decision necessary for now
- With a Nix PR to set
- Merged treewide: CUDA reformat with `nixfmt-rfc-style` by ConnorBaker · Pull Request #299578 · NixOS/nixpkgs · GitHub
- Decided on Change licensing to remove individuals (companies?) · Issue #169 · NixOS/nixfmt · GitHub
- Closed Riscv64 support? · Issue #168 · NixOS/nixfmt · GitHub
- Merged flake: add existing-but-missing devShells.default by toastal · Pull Request #181 · NixOS/nixfmt · GitHub
- Wrote down how the nixfmt/Nixpkgs CI action can be implemented: Test on Nixpkgs · Issue #163 · NixOS/nixfmt · GitHub
- @tomberek interested in working on this