-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
pre-commit, editorconfig, nixpkgs-fmt? #7634
Labels
contributor-experience
Developer experience for Nix contributors
Comments
See #6721 for C++ formatting. |
We could start by excluding existing files from the C++ formatter, in order not to hinder existing PRs with unnecessary conflicts. |
7 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
Saw some unproductive comments about formatting today. Nothing too bad, but we could prevent this.
Describe the solution you'd like
nixpkgs-fmt
pre-commit-hooks.nix
configuration that installs the hook inshellHook
.git-blame-ignore-revs
file, now also supported by GitHub web appDescribe alternatives you've considered
Carry on.
Additional context
I think we can just vote among the Nix Team and I'll be happy either way.
Priorities
Add 👍 to issues you find important.
The text was updated successfully, but these errors were encountered: