Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add nixfmt as default formatter for nix #10803

Open
TadoTheMiner opened this issue May 22, 2024 · 3 comments · May be fixed by #11001
Open

Add nixfmt as default formatter for nix #10803

TadoTheMiner opened this issue May 22, 2024 · 3 comments · May be fixed by #11001
Labels
A-language-support Area: Support for programming/text languages C-enhancement Category: Improvements

Comments

@TadoTheMiner
Copy link

Nix currently doesn't have a default formater so I think it should have configured nixfmt as a default formatter. This is a single line change
@TadoTheMiner TadoTheMiner added the C-enhancement Category: Improvements label May 22, 2024
@t56k
Copy link

t56k commented May 22, 2024

A lot of folk in Nix use Alejandra, also I think nixfmt has been deprecated

@Locorock
Copy link

A lot of folk in Nix use Alejandra, also I think nixfmt has been deprecated

last commit was last week so i doubt it. also, this NixOS/rfcs#166

@koalazub
Copy link

koalazub commented May 23, 2024 •
edited
Loading

Looks like it's already merged into master as well: NixOS/rfcs#166

Makes sense to have it as a default and to overwrite if you wanted a different one

edit: didn't realise you referenced the 'take two' of the rfc

@kirawi kirawi added the A-language-support Area: Support for programming/text languages label May 26, 2024
@notnmeyer notnmeyer linked a pull request Jun 20, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-language-support Area: Support for programming/text languages C-enhancement Category: Improvements
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

set nixfmt as the default formatter for nix notnmeyer/helix
5 participants
@Locorock @koalazub @t56k @kirawi @TadoTheMiner