2025-01-07
Attendees: @infinisil @das_j
Notes
- Don't move trailing comments in let anymore by piegamesde · Pull Request #270 · NixOS/nixfmt · GitHub
- Reviewed, looking good, merged
- Discussed a problem/solution for the upcoming full treewide reformat: Reformat Nixpkgs · Issue #322520 · NixOS/nixpkgs · GitHub
- The most-complained-about issue is Appending an optional string / array causes the entire expression to be reformatted · Issue #228 · NixOS/nixfmt · GitHub, can we resolve it?
Discussed: Appending an optional string / array causes the entire expression to be reformatted · Issue #228 · NixOS/nixfmt · GitHub - We have removed recursive mode because treefmt exists. Would be good to have a basic mode for Improving flake formatting experience · Issue #273 · NixOS/nixfmt · GitHub though. Discussed: Improving flake formatting experience · Issue #273 · NixOS/nixfmt · GitHub
- Would not come with the release builds, so we should probably drop those and encourage everybody to use the Nixpkgs version (or building from source)
- Can close Build releases for arm and darwin? · Issue #205 · NixOS/nixfmt · GitHub
- @das_j: Note that we can’t specify a specific nixfmt version anymore
- Closed Multithreading and performance optimization · Issue #211 · NixOS/nixfmt · GitHub because we maintain the decision to defer to treefmt
- Would not come with the release builds, so we should probably drop those and encourage everybody to use the Nixpkgs version (or building from source)
- What is necessary for a stable 1.0.0 release? Discussed here: Stable release · Issue #272 · NixOS/nixfmt · GitHub
2024-12-10
Attendees: @infinisil @piegames @0x4A6F Sergey + ~10 others
Notes
Done the first treewide reformat: