Run lint only in PR flow and use super-linter

Author: vgarvardt

.golangci.yml renamed to .github/linters/.golangci.yml

@@ -3,11 +3,11 @@ run:
   timeout: 5m
 linters:
   disable-all: true
-  fast: false
   enable:
     - gofmt
-    - golint
+    - revive
     - goimports
+  fast: false
 linters-settings:
   gofmt:
     simplify: false

.github/workflows/pr-lint.yml

@@ -0,0 +1,29 @@
+name: "PR: Lint"
+
+concurrency:
+  group: pr-lint-${{ github.head_ref }}
+  cancel-in-progress: true
+
+on:
+  pull_request:
+    branches: [ master, v3 ]
+
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+        with:
+          # Full git history is needed to get a proper list of changed files within `super-linter`
+          fetch-depth: 0
+
+      - name: Lint Code Base
+        uses: github/super-linter@v4
+        env:
+          VALIDATE_ALL_CODEBASE: false
+          DEFAULT_BRANCH: master
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/testing.yml

@@ -7,19 +7,10 @@ on:
     branches: [ master, v3 ]
 
 jobs:
-  lint:
-    name: Lint
-    runs-on: ubuntu-latest
-    steps:
-      - name: Check out code
-        uses: actions/checkout@v2
-      - name: golangci-lint
-        uses: golangci/golangci-lint-action@v2
-
   test:
     name: Test
     runs-on: ubuntu-latest
-    needs: [lint]
+    needs: [ lint ]
 
     services:
       postgres:

token_store.go

@@ -68,7 +68,7 @@ func NewTokenStore(adapter pgAdapter.Adapter, options ...TokenStoreOption) (*Tok
 	return store, err
 }
 
-// Close close the store
+// Close closes the store
 func (s *TokenStore) Close() error {
 	if !s.gcDisabled {
 		s.ticker.Stop()