refactor: improve taskfile & linters

Author: Utku Ozdemir

.golangci.yml

@@ -3,12 +3,16 @@ linters:
   enable-all: true
   disable:
     # deprecated:
+    - deadcode
     - exhaustivestruct
     - golint
+    - ifshort
     - interfacer
     - maligned
+    - nosnakecase
     - scopelint
-    - ifshort
+    - structcheck
+    - varcheck
     # not useful
     - exhaustruct
     # todo: consider enabling:
@@ -19,87 +23,11 @@ linters:
 
 linters-settings:
   gci:
-    local-prefixes: github.com/utkuozdemir/pv-migrate
+    sections:
+      - standard
+      - default
+      - prefix(github.com/utkuozdemir/pv-migrate)
   goimports:
     local-prefixes: github.com/utkuozdemir/pv-migrate
   goconst:
     ignore-tests: true
-
-
-#
-#
-## https://golangci-lint.run/usage/linters/
-#linters:
-#  enable:
-#    - asciicheck
-#    - bidichk
-#    - containedctx
-#    - contextcheck
-#    - cyclop
-#    - decorder
-#    - dogsled
-#    - dupl
-#    - durationcheck
-#    - errname
-#    - errorlint
-#    - exhaustive
-#    - exportloopref
-#    - forbidigo
-#    - forcetypeassert
-#    - funlen
-#    - gci
-##    - gochecknoglobals
-#    - gochecknoinits
-#    - gocognit
-#    - goconst
-#    - gocritic
-#    - gocyclo
-#    - godot
-#    - godox
-#    - goerr113
-#    - gofumpt
-#    - goheader
-##    - goimports # inconsistent with gci
-#    - gomnd
-#    - gomoddirectives
-##    - gomodguard # no need
-#    - goprintffuncname
-#    - gosec
-#    - grouper
-#    - ifshort
-#    - importas
-##    - ireturn # TODO revisit
-#    - lll
-#    - maintidx
-#    - makezero
-#    - misspell
-#    - nakedret
-#    - nestif
-#    - nilerr
-#    - nilnil
-#    - nlreturn
-#    - noctx
-#    - nolintlint
-#    - paralleltest
-#    - prealloc
-#    - predeclared
-#    - revive
-#    - rowserrcheck
-##    - sqlclosecheck
-#    - stylecheck
-#    - tagliatelle
-#    - tenv
-##    - testpackage # TODO: visit
-#    - thelper
-#    - tparallel
-#    - unconvert
-#    - unparam
-#    - varnamelen
-#    - wastedassign
-#    - whitespace
-##    - wrapcheck # TODO: visit
-#    - wsl
-#
-#linters-settings:
-#  goconst:
-#    ignore-tests: true

Taskfile.yml

@@ -3,9 +3,12 @@ version: "3"
 tasks:
   fmt:
     desc: format code
+    vars:
+      MODULE:
+        sh: head -n 1 go.mod | cut -d' ' -f2
     cmds:
       - go mod tidy
-      - gci write -s standard -s default -s "prefix(github.com/utkuozdemir/pv-migrate)" .
+      - gci write -s standard -s default -s "prefix({{.MODULE}})" .
       - gofumpt -w -l .
 
   lint:
@@ -15,18 +18,32 @@ tasks:
       - helm lint helm/pv-migrate
 
   update-chart:
-    desc: Update the Helm chart
+    desc: update the helm chart
     cmds:
       - helm-docs -c helm/pv-migrate/
       - helm package helm/pv-migrate/
       - mv pv-migrate-*.tgz internal/migrator/helm-chart.tgz
 
+  clean:
+    desc: clean
+    cmds:
+      - rm -rf {{.ROOT_DIR}}/dist/
+
+  build:
+    desc: build
+    cmds:
+      - goreleaser build --snapshot --rm-dist --single-target
+
   release:
-    desc: Create a new tag
+    desc: release
     vars:
-      NEXT:
-        sh: svu n
+      NUM_LAST_TAGS: 3
+      LAST_TAGS:
+        sh: git tag --sort=-version:refname | head -n {{.NUM_LAST_TAGS}} | xargs echo
+      NEXT_TAG:
+        sh: svu next
     cmds:
-      - git tag {{.NEXT}}
-      - echo {{.NEXT}}
-      - git push origin {{.NEXT}}
+      - "echo Last {{.NUM_LAST_TAGS}} tags: {{.LAST_TAGS}}"
+      - "echo Next tag: {{.NEXT_TAG}}"
+      - git tag -a {{.NEXT_TAG}} -m "Release {{.NEXT_TAG}}"
+      - git push origin {{.NEXT_TAG}}