Merge pull request #77 from bschaatsbergen/add-golangci

.github/workflows/ci.yaml

@@ -11,9 +11,12 @@ jobs:
         uses: actions/setup-go@v4
         with:
           go-version-file: go.mod
-      - name: Generate
-        run: go generate -v ./...
+          cache: true
       - name: Build
         run: go build -v ./...
-      - name: Test
+      - name: Run linters
+        uses: golangci/golangci-lint-action@v3
+        with:
+          version: latest
+      - name: Run tests
         run: go test -v ./...

.golangci.yml

@@ -0,0 +1,38 @@
+# Visit https://golangci-lint.run/ for usage documentation and information on other useful linters
+issues:
+  max-per-linter: 0
+  max-same-issues: 0
+
+  exclude-rules:
+    # disable funlen for test funcs
+    - source: "^func Test"
+      linters:
+        - funlen
+
+linters:
+  disable-all: true
+  enable:
+    - bidichk
+    - durationcheck
+    - decorder
+    - dogsled
+    - errcheck
+    - exportloopref
+    - forcetypeassert
+    - funlen
+    - godot
+    - godox
+    - gofmt
+    - gosimple
+    - goconst
+    - ineffassign
+    - makezero
+    - misspell
+    - nilerr
+    - predeclared
+    - staticcheck
+    - tenv
+    - unconvert
+    - unparam
+    - unused
+    - vet

cmd/root.go

@@ -17,7 +17,11 @@ var (
 		Short:   "cidr - CLI to perform various actions on CIDR ranges",
 		Version: version, // The version is set during the build by making using of `go build -ldflags`
 		Run: func(cmd *cobra.Command, args []string) {
-			cmd.Help()
+			err := cmd.Help()
+			if err != nil {
+				fmt.Println(err)
+				os.Exit(1)
+			}
 		},
 	}
 )