test: Add and clean up lint checks (#115)

.github/workflows/build.yml

@@ -40,11 +40,26 @@ jobs:
         with:
           go-version: ${{ matrix.go }}
 
-      #- name: golangci-lint
-      #  uses: golangci/golangci-lint-action@v3
-      #  if: github.event_name == 'pull_request'
-      #  with:
-      #    only-new-issues: true
+      - name: golangci-lint (gokrb5)
+        uses: golangci/golangci-lint-action@v3
+        if: github.event_name == 'pull_request'
+        with:
+          only-new-issues: true
+
+      - name: golangci-lint (apcera)
+        uses: golangci/golangci-lint-action@v3
+        if: github.event_name == 'pull_request'
+        with:
+          only-new-issues: true
+          args: --build-tags apcera
+
+      - name: golangci-lint (SSPI)
+        uses: golangci/golangci-lint-action@v3
+        if: github.event_name == 'pull_request'
+        with:
+          only-new-issues: true
+        env:
+          GOOS: windows
 
       - name: Install Kerberos client
         run: |

.golangci.yaml

@@ -0,0 +1,87 @@
+---
+issues:
+  exclude-use-default: false
+linters:
+  disable-all: true
+  enable:
+    - asasalint
+    - asciicheck
+    - bidichk
+    - bodyclose
+    - containedctx
+    - contextcheck
+    - cyclop
+    - decorder
+    - dogsled
+    - dupl
+    - dupword
+    - durationcheck
+    - errcheck
+    - errchkjson
+    - errname
+    - errorlint
+    - execinquery
+    - exhaustive
+    - exportloopref
+    - forbidigo
+    - forcetypeassert
+    - funlen
+    - gci
+    - gochecknoglobals
+    - gochecknoinits
+    - gocognit
+    - goconst
+    - gocritic
+    - gocyclo
+    - godot
+    - gofmt
+    - gofumpt
+    - goheader
+    - goimports
+    - gomoddirectives
+    - gomodguard
+    - goprintffuncname
+    - gosec
+    - gosimple
+    - govet
+    - grouper
+    - importas
+    - ineffassign
+    - interfacebloat
+    - lll
+    - loggercheck
+    - maintidx
+    - makezero
+    - misspell
+    - nakedret
+    - nestif
+    - nilerr
+    - nilnil
+    - nlreturn
+    - noctx
+    - nolintlint
+    - nosprintfhostport
+    - paralleltest
+    - prealloc
+    - predeclared
+    - promlinter
+    - reassign
+    - revive
+    - rowserrcheck
+    - sqlclosecheck
+    - staticcheck
+    - stylecheck
+    - tagliatelle
+    - tenv
+    - testableexamples
+    - testpackage
+    - thelper
+    - tparallel
+    - typecheck
+    - unconvert
+    - unparam
+    - unused
+    - usestdlibvars
+    - wastedassign
+    - whitespace
+    - wsl

.pre-commit-config.yaml

@@ -0,0 +1,9 @@
+repos:
+  - repo: https://github.com/commitizen-tools/commitizen
+    rev: v3.5.3
+    hooks:
+      - id: commitizen
+  - repo: https://github.com/golangci/golangci-lint
+    rev: v1.54.1
+    hooks:
+      - id: golangci-lint