Add golangci-lint configuration and fix linting issues (#97)

Add golangci-lint using official golangci action and fix linting issues

.github/workflows/main.yml

@@ -16,14 +16,20 @@ jobs:
           go-version: 1.18
 
       - name: Checkout code
-        uses: actions/checkout@master
+        uses: actions/checkout@v2
 
-      - name: run tests
+      - name: Run golangci-lint
+        uses: golangci/golangci-lint-action@v3
+        with:
+          version: v1.49.0
+          args: --config .golangci.yml
+
+      - name: Run tests
         run: |
-          go test -coverprofile=coverage.txt -covermode=atomic ./...       
+          go test -coverprofile=coverage.txt -covermode=atomic ./...
 
       - name: Publish coverage
         uses: codecov/[email protected]
         with:
-          token: ${{secrets.CODECOV_TOKEN}}
+          token: ${{ secrets.CODECOV_TOKEN }}
           file: ./coverage.txt

.golangci.yml

@@ -0,0 +1,37 @@
+linters:
+  enable:
+    - govet
+    - errcheck
+    - staticcheck
+    - unused
+    - gofmt
+    - goimports
+    - gocritic
+
+linters-settings:
+  govet:
+    check-shadowing: true
+  gocritic:
+    check:
+      - range-loop-ref
+      - ifElseChain
+      - typeSwitchVar
+      - wrapperFunc
+
+run:
+  timeout: 5m
+  tests: true
+  skip-dirs:
+    - vendor
+
+issues:
+  max-issues-per-linter: 0
+  max-same-issues: 0
+  exclude-use-default: false
+  include:
+    - EXC0001
+
+exclude-rules:
+  - path: _test\.go
+    linters:
+      - errcheck

bot_test.go

@@ -46,7 +46,11 @@ func (s *serverMock) handler(rw http.ResponseWriter, req *http.Request) {
 	if errReadBody != nil {
 		panic(errReadBody)
 	}
-	defer req.Body.Close()
+	defer func() {
+		if err := req.Body.Close(); err != nil {
+			panic(err)
+		}
+	}()
 
 	hook, okHook := s.hooks[req.URL.String()]
 	if okHook {

build_request_form_test.go

@@ -11,7 +11,7 @@ import (
 
 func assertFormData(t *testing.T, data, expect string) {
 	if !strings.Contains(expect, "\r\n") {
-		expect = strings.Replace(expect, "\n", "\r\n", -1)
+		expect = strings.ReplaceAll(expect, "\n", "\r\n")
 	}
 
 	if data != expect {
@@ -60,7 +60,9 @@ func Test_buildRequestForm(t *testing.T) {
 		t.Error(errBuild)
 		return
 	}
-	form.Close()
+	if err := form.Close(); err != nil {
+		t.Errorf("failed to close form: %v", err)
+	}
 
 	expect := `--XXX
 Content-Disposition: form-data; name="string"

raw_request.go

@@ -6,6 +6,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"io"
+	"log"
 	"mime/multipart"
 	"net/http"
 	"reflect"
@@ -67,7 +68,11 @@ func (b *Bot) rawRequest(ctx context.Context, method string, params any, dest an
 	if errDo != nil {
 		return fmt.Errorf("error do request for method %s, %w", method, errDo)
 	}
-	defer resp.Body.Close()
+	defer func() {
+		if err := resp.Body.Close(); err != nil {
+			log.Printf("failed to close response body: %v", err)
+		}
+	}()
 
 	body, errReadBody := io.ReadAll(resp.Body)
 	if errReadBody != nil {