fix: workflow (#89)

* fix: workflow
* fixing mistake
* fix: new linter issues

Author: butuzov

.github/workflows/main.yaml

@@ -15,47 +15,34 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        go:
-          - "1.24"
-          - "1.25"
-          - "1.26"
+        go: [oldstable, stable]
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - name: Install Go
-        uses: actions/setup-go@v5
+        uses: actions/setup-go@v6
         with:
           go-version: ${{ matrix.go }}
 
-      - uses: actions/cache@v4
-        if: ${{ !env.ACT }}
-        with:
-          path: ~/go/pkg/mod
-          key: ${{ runner.os }}-go-${{ matrix.golang }}-${{ hashFiles('**/go.sum') }}
-          restore-keys: |
-            ${{ runner.os }}-go-${{ matrix.golang }}-
-
       - run: make build
       - run: make tests
 
       - name: Install goveralls
-        if: matrix.go == '1.26'
+        if: matrix.go == 'stable'
         run: go install github.com/mattn/goveralls@latest
 
       - name: Coverage - Sending Report to Coveral
-        if: matrix.go == '1.26'
+        if: matrix.go == 'stable'
         env:
           COVERALLS_TOKEN: ${{ secrets.github_token }}
         run: goveralls -coverprofile=coverage.cov -service=github
 
-  GolangCI_Linter:
+  GolangCILinter:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
       - name: lint
-        uses: golangci/golangci-lint-action@v6.1.1
+        uses: golangci/golangci-lint-action@v9
         with:
           version: latest
-          skip-cache: true
-          args: --exclude-dirs "^(cmd|testdata)"

.github/workflows/release.yaml

@@ -5,34 +5,22 @@ on:
     tags:
       - v*
 
-env:
-  GO_VERSION: "1.25"
-
 jobs:
-  GoReleaser:
+  GoReeleaser:
     name: release
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
         with:
           fetch-depth: 0
 
       - uses: actions/setup-go@v5
         with:
-          go-version: ${{ env.GO_VERSION }}
-
-      - uses: actions/cache@v4
-        if: ${{ !env.ACT }}
-        with:
-          path: ~/go/pkg/mod
-          key: ${{ runner.os }}-go-${{ matrix.golang }}-${{ hashFiles('**/go.sum') }}
-          restore-keys: |
-            ${{ runner.os }}-go-${{ matrix.golang }}-
+          go-version: stable
 
       - name: GoReleaser
         uses: goreleaser/goreleaser-action@v6
         with:
-          version: latest
           args: release --clean
         env:
           GITHUB_TOKEN: ${{ secrets.GORELEASER_GITHUB_TOKEN }}

.golangci.yaml

@@ -0,0 +1,26 @@
+version: "2"
+
+linters:
+  enable:
+    - nolintlint
+    - testifylint
+  settings:
+    nolintlint:
+      require-specific: true
+      allow-unused: false
+  exclusions:
+    presets:
+      - comments
+      - common-false-positives
+      - std-error-handling
+
+formatters:
+  enable:
+    - gci
+    - gofumpt
+  settings:
+    gci:
+      sections:
+        - standard
+        - default
+        - prefix(github.com/butuzov/mirror)

analyzer.go

@@ -5,11 +5,11 @@ import (
 	"go/ast"
 	"strings"
 
-	"github.com/butuzov/mirror/internal/checker"
-
 	"golang.org/x/tools/go/analysis"
 	"golang.org/x/tools/go/analysis/passes/inspect"
 	"golang.org/x/tools/go/ast/inspector"
+
+	"github.com/butuzov/mirror/internal/checker"
 )
 
 func NewAnalyzer() *analysis.Analyzer {

cmd/internal/tests/main.go

@@ -36,7 +36,7 @@ func main() {
 		tests = append(tests, generateTests("regexp", mirror.RegexpFunctions)...)
 		tests = append(tests, generateTests("regexp", mirror.RegexpRegexpMethods)...)
 
-		GenerateTestFile(filepath.Join(testdata, "regexp.go"), "regexp", tests)
+		_ = GenerateTestFile(filepath.Join(testdata, "regexp.go"), "regexp", tests)
 	}
 
 	{ // strings

cmd/internal/tests/support.go

@@ -158,18 +158,19 @@ func generateTests(pkgName string, list []checker.Violation) []string {
 				pkgInTest := pkg
 				preCondition := test.Generate.PreCondition
 				if test.Generate.PreCondition != "" {
-					pkgInTest = strings.Trim(strings.Split(test.Generate.PreCondition, ":=")[0], " ")
+					pkgInTest = strings.Trim(strings.Split(
+						test.Generate.PreCondition, ":=")[0], " ")
 
 					alt := pkg + "."
 					if strings.Trim(pkg, " ") == "" {
 						alt = ""
 					}
 
-					preCondition = strings.Replace(preCondition, pkgName+".", alt, -1)
+					preCondition = strings.ReplaceAll(preCondition, pkgName+".", alt)
 
 				}
 
-				templates.ExecuteTemplate(&buf, "case.tmpl", TestCase{
+				_ = templates.ExecuteTemplate(&buf, "case.tmpl", TestCase{
 					Arguments: []string{},
 					Returns:   GenReturnElements(len(test.Generate.Returns)),
 					Package:   pkgInTest,

cmd/mirror/main.go

@@ -1,8 +1,9 @@
 package main
 
 import (
-	"github.com/butuzov/mirror"
 	"golang.org/x/tools/go/analysis/singlechecker"
+
+	"github.com/butuzov/mirror"
 )
 
 func main() {

internal/checker/checker.go

@@ -109,7 +109,7 @@ func (c *Checker) callConverts(ce *ast.CallExpr) bool {
 
 // register violations.
 func (c *Checker) register(violations []Violation) {
-	for _, v := range violations { // nolint: gocritic
+	for _, v := range violations { //nolint: gocritic
 		c.Violations = append(c.Violations, v)
 		if len(v.Struct) > 0 {
 			c.registerIdxPer(v.Package + "." + v.Struct)

internal/checker/imports_test.go

@@ -82,7 +82,8 @@ func TestImports(t *testing.T) {
 
 			// assert
 			if len(testImports["a.go"]) != test.importsLen {
-				t.Errorf("Imports len not match: got(%d) vs want(%d)", len(testImports["a.go"]), test.importsLen)
+				t.Errorf("Imports len not match: got(%d) vs want(%d)",
+					len(testImports["a.go"]), test.importsLen)
 			}
 
 			for k, v := range test.hasImports {