Merge pull request #5 from alk-xchassin/master

Add the linter to the CI + the git diff

.github/workflows/go-test.yml

@@ -2,28 +2,35 @@ name: Test
 
 on:
   push:
-    branches: [ master ]
+    branches: [master]
   pull_request:
-    branches: [ master ]
+    branches: [master]
 
 jobs:
-
   build:
     name: Test
     runs-on: ubuntu-latest
     steps:
+      - name: Set up Go
+        uses: actions/setup-go@v2
+        with:
+          go-version: ^1.14
+        id: go
+
+      - name: Check out code into the Go module directory
+        uses: actions/checkout@v2
 
-    - name: Set up Go 1.x
-      uses: actions/setup-go@v2
-      with:
-        go-version: ^1.14
-      id: go
+      - name: Get dependencies
+        run: go mod download
 
-    - name: Check out code into the Go module directory
-      uses: actions/checkout@v2
+      - name: Test
+        run: go test -vet=off ./...
 
-    - name: Get dependencies
-      run: go mod download
+      - name: Lint
+        uses: golangci/golangci-lint-action@v2
+        with:
+          version: v1.30
 
-    - name: Test
-      run: go test -vet=off ./...
+      - name: Check diffs
+        # --name-only first for debugging
+        run: git diff --name-only && git diff --quiet

.github/workflows/release.yml

@@ -0,0 +1,30 @@
+name: Release
+
+on:
+  push:
+    tags:
+      - "*"
+
+jobs:
+  build:
+    name: Release
+    runs-on: ubuntu-latest
+    steps:
+      - name: Set up Go
+        uses: actions/setup-go@v2
+        with:
+          go-version: ^1.14
+        id: go
+
+      - name: Check out code into the Go module directory
+        uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+
+      - name: Run GoReleaser
+        uses: goreleaser/goreleaser-action@v2
+        with:
+          version: latest
+          args: release --rm-dist
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.golangci.yml

@@ -0,0 +1,13 @@
+run:
+  deadline: 8m
+  build-tags:
+    - codeanalysis
+
+linters:
+  enable:
+    - gosec # (gas) Inspects source code for security problems
+    - nolintlint # Reports ill-formed or insufficient nolint directives
+    - unconvert # Remove unnecessary type conversions
+    - whitespace # Tool for detection of leading and trailing whitespace
+    - gofumpt # Gofumpt checks whether code was gofumpt-ed
+    - prealloc # Finds slice declarations that could potentially be preallocated

.goreleaser.yml

@@ -0,0 +1,6 @@
+project_name: mylib-go
+build:
+  skip: true
+release:
+  github:
+  prerelease: auto

example/nodes/math.go

@@ -15,7 +15,7 @@ func Adder(a, b int) (sum int) { return a + b }
 func Multiplier(a, b int) (product int) { return a * b }
 
 func RandomIntProducer() (n int) {
-	return rand.New(rand.NewSource(time.Now().UnixNano())).Intn(100)
+	return rand.New(rand.NewSource(time.Now().UnixNano())).Intn(100) //nolint:gosec // not really critical
 }
 
 // IntAggregator aggregates `list` using `reducer.