Don't format files in the pre-push check

Signed-off-by: Joonas Rautiola <joonas.rautiola@unikie.com>
tiiuae · Dec 19, 2023 · 43fe4c0 · 43fe4c0
1 parent dd4dbb6
commit 43fe4c0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/nix/devshell.nix b/nix/devshell.nix
@@ -5,6 +5,7 @@
   perSystem = {pkgs, ...}: {
     devShells.default = pkgs.mkShell {
       packages = with pkgs; [
+        alejandra
         azure-cli
         git
         nix

diff --git a/tasks.py b/tasks.py
@@ -495,9 +495,10 @@ def pre_push(c: Any) -> None:
     if not ret:
         LOG.warning("Run `terraform fmt -recursive` locally to fix formatting")
         sys.exit(1)
-    cmd = "nix fmt"
+    cmd = "nix fmt -- --fail-on-change"
     ret = exec_cmd(cmd, raise_on_error=False)
     if not ret:
+        LOG.warning("Run `nix fmt` locally to fix formatting")
         sys.exit(1)
     cmd = "nix flake check -v"
     ret = exec_cmd(cmd, raise_on_error=False)