Merge pull request cachix#309 from Enzime/fix/nil

nil: Fix hook only failing if last file fails
semnix · May 29, 2023 · ca2fdbf · ca2fdbf
2 parents 8e2c1a5 + da69577
commit ca2fdbf
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/modules/hooks.nix b/modules/hooks.nix
@@ -731,9 +731,20 @@ in
           entry =
             let
               script = pkgs.writeShellScript "precommit-nil" ''
+                errors=false
+                echo Checking: $@
                 for file in $(echo "$@"); do
                   ${tools.nil}/bin/nil diagnostics "$file"
+                  exit_code=$?
+
+                  if [[ $exit_code -ne 0 ]]; then
+                    echo \"$file\" failed with exit code: $exit_code
+                    errors=true
+                  fi
                 done
+                if [[ $errors == true ]]; then
+                  exit 1
+                fi
               '';
             in
             builtins.toString script;