Merge pull request #375 from snf1k/golangci-lint

Add golangci-lint

README.md

@@ -173,6 +173,7 @@ use nix
 
 - gofmt: Runs `go fmt`
 - gotest: Runs `go test`
+- [golangci-lint](https://golangci-lint.run/)
 - [govet](https://pkg.go.dev/cmd/vet)
 - [revive](https://github.com/mgechev/revive)
 - [staticcheck](https://github.com/dominikh/go-tools)

modules/hooks.nix

@@ -2208,5 +2208,24 @@ in
         description = "Runs the git-annex hook for large file support";
         entry = "${tools.git-annex}/bin/git-annex pre-commit";
       };
+
+      golangci-lint = {
+        name = "golangci-lint";
+        description = "Fast linters runner for Go.";
+        entry =
+          let
+            script = pkgs.writeShellScript "precommit-golangci-lint" ''
+              set -e
+              for dir in $(echo "$@" | xargs -n1 dirname | sort -u); do
+                ${tools.golangci-lint}/bin/golangci-lint run ./"$dir"
+              done
+            '';
+          in
+          builtins.toString script;
+        files = "\\.go$";
+        # to avoid multiple invocations of the same directory input, provide
+        # all file names in a single run.
+        require_serial = true;
+      };
     };
 }

nix/tools.nix

@@ -67,6 +67,7 @@
 , writeText
 , go
 , go-tools
+, golangci-lint
 , revive ? null
 }:
 
@@ -98,6 +99,7 @@ in
     git-annex
     go
     go-tools
+    golangci-lint
     gptcommit
     hadolint
     hindent