feat: add two new workflows (golangci-lint and test) (#7)

.github/workflows/golangci-lint.yml

@@ -0,0 +1,35 @@
+name: golangci-lint
+
+on:
+  push:
+    tags:
+      - v*
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+permissions:
+  contents: read
+
+jobs:
+  golangci:
+    name: lint
+
+    strategy:
+      matrix:
+        platform: [ ubuntu-latest ]
+        go-version: [ 1.18.x ]
+
+    runs-on: ${{ matrix.platform }}
+
+    steps:
+      - uses: actions/setup-go@v3
+        with:
+          go-version: ${{ matrix.go-version }}
+      - uses: actions/checkout@v3
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v3
+        with:
+          version: v1.46

.github/workflows/test.yml

@@ -0,0 +1,38 @@
+name: test
+
+on:
+  push:
+    tags:
+      - 'v*'
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  test:
+    strategy:
+      matrix:
+        go-version: [ 1.18.x ]
+        platform: [ ubuntu-latest ]
+
+    runs-on: ${{ matrix.platform }}
+
+    steps:
+      - name: Install Go
+        uses: actions/setup-go@v3
+        with:
+          go-version: ${{ matrix.go-version }}
+
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: go mod download
+        run: go mod download
+
+      - name: Vet
+        run: go vet ./...
+
+      - name: Test
+        run: go mod download && go test -race -coverprofile=coverage.txt -covermode=atomic ./...

.golangci.yml

@@ -0,0 +1,44 @@
+run:
+  tests: true
+
+linters:
+  disable-all: true
+  enable:
+    - bodyclose
+    - deadcode
+    - depguard
+    - dupl
+    - errcheck
+    - gocritic
+    - gofmt
+    - goimports
+    - gosimple
+    - govet
+    - ineffassign
+    - misspell
+    - nakedret
+    - prealloc
+    - staticcheck
+    - structcheck
+    - typecheck
+    - unconvert
+    - unused
+    - varcheck
+    - lll
+    - nestif
+    - thelper
+    - nonamedreturns
+
+linters-settings:
+  lll:
+    line-length: 150
+
+issues:
+  exclude-rules:
+    # Exclude some linters from running on tests files.
+    - path: _test\.go
+      linters:
+        - dupl
+    - linters:
+        - lll
+      source: "^//go:generate "

internal/andotp_test.go

@@ -2,10 +2,11 @@ package internal_test
 
 import (
 	"encoding/json"
-	"github.com/Kichiyaki/gootp/internal"
 	"testing"
 	"time"
 
+	"github.com/Kichiyaki/gootp/internal"
+
 	"github.com/stretchr/testify/assert"
 )
 

internal/otp.go

@@ -2,10 +2,11 @@ package internal
 
 import (
 	"fmt"
-	"github.com/pquerna/otp"
-	"github.com/pquerna/otp/totp"
 	"strings"
 	"time"
+
+	"github.com/pquerna/otp"
+	"github.com/pquerna/otp/totp"
 )
 
 func GenerateOTP(entry Entry) (string, error) {

main.go

@@ -2,13 +2,14 @@ package main
 
 import (
 	"fmt"
-	"github.com/Kichiyaki/gootp/internal"
-	"github.com/urfave/cli/v2"
-	"golang.org/x/term"
 	"log"
 	"os"
 	"path"
 	"syscall"
+
+	"github.com/Kichiyaki/gootp/internal"
+	"github.com/urfave/cli/v2"
+	"golang.org/x/term"
 )
 
 var Version = "development"