-
-
Notifications
You must be signed in to change notification settings - Fork 189
chore: add treefmt
config for nix files
#1653
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.
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
base: develop
Are you sure you want to change the base?
Conversation
c47d5d7
to
943f859
Compare
1de4fa9
to
943f859
Compare
with: | ||
nix_path: nixpkgs=channel:nixos-unstable | ||
- name: treefmt | ||
run: nix-shell -p treefmt nixpkgs-fmt --run "treefmt --ci" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC nix fmt
doesn't have a "check" mode, so here I call treefmt --ci
directly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yvan-sraka this looks great. Can you please add a bit in the nix/docs section to give some clues to people who are not as deeply embedded in Nix tooling etc how to use what was added here (maybe they need to add a plugin for their editor, anything else to know once this is merged and added)
943f859
to
f662771
Compare
f662771
to
d595e46
Compare
* add a ci job that run `treefmt` * make it `nix fmt` default formatter * add dumb simple `pre-commit` and `editorconfig`
d595e46
to
15311a9
Compare
treefmt
nix fmt
default formatterpre-commit
andeditorconfig