Improve ci action (#135)

Author: w41ter

.github/workflows/go.yml

@@ -5,12 +5,10 @@ name: Go
 
 on:
   push:
-    branches: [ "dev", "branch-2.0" ]
+    branches: [ "dev", "branch-2.0", "branch-2.1", "branch-3.0" ]
   pull_request:
-    branches: [ "dev", "branch-2.0" ]
 
 jobs:
-
   build:
     runs-on: ubuntu-latest
     steps:
@@ -21,6 +19,9 @@ jobs:
       with:
         go-version: '1.20'
 
+    - name: Format
+      run: if [ "$(gofmt -s -l . | wc -l)" -gt 0 ]; then exit 1; fi
+    
     - name: Build
       run: make
 

.github/workflows/golangci-lint.yml

@@ -0,0 +1,29 @@
+name: golangci-lint
+on:
+  push:
+    branches:
+      - main
+      - dev
+      - branch-3.0
+      - branch-2.1
+      - branch-2.0
+  pull_request:
+
+permissions:
+  contents: read
+  # Optional: allow read access to pull request. Use with `only-new-issues` option.
+  # pull-requests: read
+
+jobs:
+  golangci:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: '1.20'
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.60

Makefile

@@ -20,13 +20,7 @@ tarball_suffix := $(tag)-$(platform)
 LDFLAGS="-X 'github.com/selectdb/ccr_syncer/pkg/version.GitTagSha=$(git_tag_sha)'"
 GOFLAGS=
 
-# Check formatter, if exist gofumpt, use it
-GOFUMPT := $(shell command -v gofumpt 2> /dev/null)
-ifdef GOFUMPT
-	GOFORMAT := gofumpt -w
-else
-	GOFORMAT := go fmt
-endif
+GOFORMAT := gofmt -l -d -w
 
 # COVERAGE=ON make
 ifeq ($(COVERAGE),ON)
@@ -63,7 +57,7 @@ fmt:
 .PHONY: test
 ## test : Run test
 test:
-	$(V)go test $(shell go list ./... | grep -v github.com/selectdb/ccr_syncer/cmd | grep -v github.com/selectdb/ccr_syncer/pkg/rpc/kitex_gen/) | grep -F -v '[no test files]'
+	$(V)go test $(shell go list ./... | grep -v github.com/selectdb/ccr_syncer/cmd | grep -v github.com/selectdb/ccr_syncer/pkg/rpc/kitex_gen/)
 
 .PHONY: help
 ## help : Print help message