feat: add golangci-lint configuration with pre-commit hooks to lint code (#116)

Author: kklimonda-cl

.github/workflows/lint_pr_title.yml

@@ -21,4 +21,4 @@ jobs:
     steps:
       - uses: amannn/action-semantic-pull-request@e9fabac35e210fea40ca5b14c0da95a099eff26f # v5
         env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/pr_ci.yml

@@ -12,6 +12,18 @@ on:
       - ready_for_review
 
 jobs:
+  golangci:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: stable
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.59
   ci:
     name: Continuous integration
     runs-on: ubuntu-latest
@@ -21,15 +33,15 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: '1.21'
+          go-version: "1.22"
 
       - name: Download Go dependencies
         run: go get  ./... && go mod tidy
 
       - name: Check if all dependencies were commited
         shell: bash
         run: |
-          if [ "$(git status --porcelain)" ]; then 
+          if [ "$(git status --porcelain)" ]; then
            git status
            exit 1
           else
@@ -51,7 +63,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@cdcb36043654635271a94b9a6d1392de5bb323a7 # v5
         with:
-          go-version: '1.21'
+          go-version: "1.22"
 
       - name: Download Go dependencies
         run: go get  ./... && go mod tidy

.gitignore

@@ -18,4 +18,4 @@
 *.swp
 
 # IDE
-.idea/
+.idea/

.golangci.yaml

@@ -0,0 +1,43 @@
+linters:
+  enable:
+    - asasalint
+    - copyloopvar
+    - decorder
+    - dogsled
+    - errname
+    - exhaustive
+    - exportloopref
+    - fatcontext
+    - forcetypeassert
+    - gci
+    - ginkgolinter
+    - gofmt
+    - grouper
+    - importas
+    - makezero
+    - nakedret
+    - nilerr
+    - nilnil
+    - predeclared
+    - sloglint
+    #- wrapcheck  # this one will be useful, but we have to fix a multitude of errors
+    #- unparam  # when fixed, probably already covered by revive/gocritic
+    #- revive  # too many errors, but useful, similar to gocritic, so probably choose one
+    #- prealloc  # enable when can
+    #- nestif  # enable once we clean up the code
+    #- gofumpt  # fix all files before we switch to gofumpt
+    #- gocritic  # too many failures for now
+    #- goconst  # once we rewrite string literals to consts
+    #- gochecknoglobals  # once we rewrite terraform codegen and drop global functions
+    #- funlen  # couple of functions over 60 lines
+    #- err113  # disabled because of too many dynamic errors that don't wrap anything
+
+linters-settings:
+  gci:
+    sections:
+      - standard
+      - default
+      - prefix(github.com/paloaltonetworks/pan-os-codegen)
+issues:
+  exclude-dirs:
+    - generated/

.pre-commit-config.yaml

@@ -0,0 +1,10 @@
+repos:
+  - repo: https://github.com/golangci/golangci-lint/
+    rev: v1.59.1
+    hooks:
+      - id: golangci-lint
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v2.3.0
+    hooks:
+      - id: end-of-file-fixer
+      - id: trailing-whitespace

README.md

@@ -33,7 +33,7 @@ The code have run login in `cmd/codegen` directory, to run it with default optio
 go run cmd/codegen/main.go
 ```
 This command can be parametrizes using options:
-- `-t/-type` - operation type, default is to create both Terraform  
+- `-t/-type` - operation type, default is to create both Terraform
   - `mktp` - create only Terraform provider
   - `mksdk` - create only PAN-OS SDK
 - `config` - specify path for the config file, default is `cmd/codegen/config.yaml`
@@ -47,4 +47,4 @@ example code:
 go run cmd/codegen/main.go -t mksdk
 cd ../generated/pango
 PANOS_HOSTNAME='***' PANOS_USERNAME='***' PANOS_PASSWORD='***' go run example/main.go
-```
+```

SUPPORT.md

@@ -4,4 +4,4 @@ This template/script/solution is released “as-is”, with no warranty and no s
 supported and Palo Alto Networks may contribute its expertise at its discretion. Palo Alto Networks, including through
 its Authorized Support Centers (ASC) partners and backline support options, will not provide technical support or help
 in using or troubleshooting this template/script/solution. The underlying product used by this template/script/solution
-will still be supported in accordance with the product’s applicable support policy and the customer’s entitlements.
+will still be supported in accordance with the product’s applicable support policy and the customer’s entitlements.

assets/pango/go.mod

@@ -1,6 +1,6 @@
 module github.com/PaloAltoNetworks/pango
 
-go 1.21.4
+go 1.22
 
 require (
 	github.com/onsi/ginkgo/v2 v2.19.0

assets/terraform/go.mod

@@ -1,6 +1,6 @@
 module github.com/PaloAltoNetworks/terraform-provider-panos
 
-go 1.21.4
+go 1.22
 
 require (
 	github.com/PaloAltoNetworks/pango v0.10.3-0.20240408115758-216d8509e7cf

go.mod

@@ -10,4 +10,4 @@ require (
 	github.com/pmezard/go-difflib v1.0.0 // indirect
 )
 
-go 1.21.4
+go 1.22.5