Fix lint CI

.github/workflows/workflow.yml

@@ -7,17 +7,16 @@ jobs:
       matrix:
         os:
           - ubuntu-latest
-          - macos-latest
-          - windows-latest
+          # - macos-latest
+          # - windows-latest
         go:
           - '1.19'
-          - '1.18'
-          - '1.17'
-          - '1.16'
-          - '1.15'
+          # - '1.18'
+          # - '1.17'
+          # - '1.16'
+          # - '1.15'
     env:
       OS: ${{ matrix.os }}
-      PYTHON: '3.7'
     steps:
     - uses: actions/checkout@master
 
@@ -26,8 +25,10 @@ jobs:
       with:
         go-version: ${{ matrix.go }}
 
-    - name: Lint
-      run: make setup && make lint
+    - name: golangci-lint
+      uses: golangci/golangci-lint-action@v3
+      with:
+        version: v1.48
 
     - name: Test
       run: make test

Makefile

@@ -3,26 +3,14 @@ TEST_PACKAGES	:= $(shell go list ./... | grep -v cmd)
 COVER_PACKAGES	:= $(shell go list ./... | grep -v cmd | paste -sd "," -)
 LINTER		:= $(shell command -v gometalinter 2> /dev/null)
 
-.PHONY: setup
-
-setup:
-ifndef LINTER
-	@echo "Installing linter"
-	@go get -u github.com/alecthomas/gometalinter
-	@gometalinter --install
-endif
-
 build:
 	@go build -o ./gaper cmd/gaper/main.go
 
 ## lint: Validate golang code
+# Install it following this doc https://golangci-lint.run/usage/install/#local-installation,
+# please use the same version from .github/workflows/workflow.yml.
 lint:
-	@gometalinter \
-		--deadline=120s \
-		--line-length=120 \
-		--enable-all \
-		--disable=gochecknoinits --disable=gochecknoglobals \
-		--vendor ./...
+	@golangci-lint run
 
 test:
 	@go test -p=1 -coverpkg $(COVER_PACKAGES) \

builder.go

@@ -17,7 +17,6 @@ type Builder interface {
 type builder struct {
 	dir       string
 	binary    string
-	errors    string
 	wd        string
 	buildArgs []string
 }

gaper.go

@@ -212,9 +212,7 @@ func setupConfig(cfg *Config) error {
 	var extensions []string
 	for i := range cfg.Extensions {
 		values := strings.Split(cfg.Extensions[i], ",")
-		for _, e := range values {
-			extensions = append(extensions, e)
-		}
+		extensions = append(extensions, values...)
 	}
 	cfg.Extensions = extensions
 

watcher_test.go

@@ -104,14 +104,19 @@ func TestWatcherWatchChange(t *testing.T) {
 	time.Sleep(time.Millisecond * 500)
 
 	// update hidden files and dirs to check builtin hidden ignore is working
-	os.Chtimes(hiddenfile1, time.Now(), time.Now())
-	os.Chtimes(hiddenfile2, time.Now(), time.Now())
+	err = os.Chtimes(hiddenfile1, time.Now(), time.Now())
+	assert.Nil(t, err, "chtimes error")
+
+	err = os.Chtimes(hiddenfile2, time.Now(), time.Now())
+	assert.Nil(t, err, "chtimes error")
 
 	// update testfile first to check ignore is working
-	os.Chtimes(testfile, time.Now(), time.Now())
+	err = os.Chtimes(testfile, time.Now(), time.Now())
+	assert.Nil(t, err, "chtimes error")
 
 	time.Sleep(time.Millisecond * 500)
-	os.Chtimes(mainfile, time.Now(), time.Now())
+	err = os.Chtimes(mainfile, time.Now(), time.Now())
+	assert.Nil(t, err, "chtimes error")
 
 	select {
 	case event := <-w.Events():