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.

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

Add nix fmt hook #495

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions modules/hooks.nix
Original file line number Diff line number Diff line change
Expand Up @@ -2894,6 +2894,13 @@ lib.escapeShellArgs (lib.concatMap (ext: [ "--ghc-opt" "-X${ext}" ]) hooks.ormol
builtins.toString script;
files = "\\.nix$";
};
nix-fmt =
{
name = "nix-fmt";
description = "Format Nix files using the formatter specified in your flake.";
package = tools.nix;
entry = "${hooks.nix-fmt.package}/bin/nix fmt";
};
# nixfmt was renamed to nixfmt-classic.
# The hook has been deprecated to free up the name for when the new RFC-style nixfmt becomes stable.
nixfmt = nixfmt-classic;
Expand Down
2 changes: 2 additions & 0 deletions nix/tools.nix
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@
, mdl
, mdsh
, nil
, nix
, nixfmt
, nixfmt-classic ? null
, nixfmt-rfc-style ? null
Expand Down Expand Up @@ -134,6 +135,7 @@ in
mdl
mdsh
nil
nix
nixpkgs-fmt
ormolu
pre-commit-hook-ensure-sops
Expand Down