all: Update workflow actions; set v1.55.2 as lint version (#10)

.github/workflows/lint.yml

@@ -1,17 +1,17 @@
 name: Run linters
 
-on: [push]
+on: [push, pull_request]
 
 jobs:
   revive-lint:
     name: Revive
     runs-on: ubuntu-latest
     steps:
       - name: Checkout the code
-        uses: actions/checkout@v1
+        uses: actions/checkout@v4
 
       - name: Run Revive
-        uses: docker://morphy/revive-action:v1
+        uses: docker://morphy/revive-action:v2
         with:
           config: .revive.toml
         env:
@@ -22,10 +22,11 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Check out code into the Go module directory
-        uses: actions/checkout@v1
+        uses: actions/checkout@v4
       - name: Run GolangCI-Lint
-        uses: docker://matousdz/golangci-lint-action:v1.0.0
+        uses: golangci/golangci-lint-action@v3
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         with:
           config: .golangci.yml
+          version: v1.55.2

.github/workflows/test.yml

@@ -1,18 +1,18 @@
 name: Run tests
 
-on: [push]
+on: [push, pull_request]
 
 jobs:
   tests:
     name: Tests
     runs-on: ubuntu-latest
     steps:
       - name: Checkout the code
-        uses: actions/checkout@v1
+        uses: actions/checkout@v4
 
       - name: Run tests
-        uses: actions/setup-go@v1
+        uses: actions/setup-go@v5
         with:
-          go-version: '1.13'
+          go-version: '1.18'
 
-      - run: go test ./...
+      - run: go test -race -shuffle=on ./...

.golangci.yml

@@ -1,10 +1,4 @@
 linters-settings:
-  depguard:
-    list-type: blacklist
-    include-go-root: true
-    packages:
-      # we are using "github.com/json-iterator/go" instead of json encoder from stdlib
-      - "encoding/json"
   dupl:
     threshold: 100
   gocritic:
@@ -38,27 +32,37 @@ linters-settings:
 linters:
   enable-all: true
   disable:
+    - depguard
     # prealloc is not recommended by `golangci-lint` developers.
     - prealloc
     - gochecknoglobals
 
+    # deprecated
+    - maligned
+    - exhaustivestruct
+    - nosnakecase
+    - scopelint
+    - structcheck
+    - ifshort
+    - varcheck
+    - deadcode
+    - golint
+    - interfacer
+
 issues:
   exclude-rules:
     - path: _test\.go
       linters:
+        - exhaustruct
         - goconst
         - dupl
 
-    - path: fixtures
-      linters:
-        - gocritic
-        - varcheck
-        - deadcode
-        - unused
-
 run:
   modules-download-mode: readonly
 
+  skip-dirs:
+    - "fixtures"
+
 # output configuration options
 output:
   # colored-line-number|line-number|json|tab|checkstyle|code-climate, default is "colored-line-number"

.revive.toml

@@ -1,5 +1,6 @@
 ignoreGeneratedHeader = false
 severity = "warning"
+exclude = ["./fixtures/..."]
 
 # confidence <= 0.2 generate a lot of errors from package-comments rule. It marks files that do not contain
 # package-level comments as a warning irrespective of existing package-level coment in one file.

fixtures/01/example1.go

@@ -3,9 +3,8 @@ package main
 
 import "fmt"
 
-// This comment is associated with the hello constant.x
-//
 //revive:disable
+// This comment is associated with the hello constant.x
 const hello = "Hello, World!" // line comment 1
 
 // This comment is associated with the foo variable.

godox.go

@@ -1,3 +1,5 @@
+// Package godox is a linter that scans Go code for comments containing certain keywords
+// (like TODO, BUG, FIXME) which typically indicate areas that require attention.
 package godox
 
 import (