Skip to content

Commit

Permalink
fix: add golangci-lint setup action and fix lint issue (#592)
Browse files Browse the repository at this point in the history
Co-authored-by: Neemias Almeida <ndalmeida@bol.com>
  • Loading branch information
ndajr and Neemias Almeida authored May 28, 2024
1 parent f673320 commit e3b6eaf
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,14 @@ jobs:
uses: actions/setup-go@v4
with:
go-version: ^1.22

- name: Build
run: make build
- name: Install golangci-lint
uses: golangci/golangci-lint-action@v6
with:
version: v1.59
- name: Install dependency
run: if [ $(uname) == "Darwin" ]; then brew install gnu-sed ;fi
- name: Build
run: make build
- name: Run unit tests
run: go install github.com/go-delve/delve/cmd/dlv@latest && go test ./... -v -covermode=count -coverprofile=coverage.txt
- name: Upload Coverage report to CodeCov
Expand Down
2 changes: 1 addition & 1 deletion runner/proxy.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ func NewProxy(cfg *cfgProxy) *Proxy {
Addr: fmt.Sprintf(":%d", cfg.ProxyPort),
},
client: &http.Client{
CheckRedirect: func(req *http.Request, via []*http.Request) error {
CheckRedirect: func(_ *http.Request, _ []*http.Request) error {
return http.ErrUseLastResponse
},
},
Expand Down

0 comments on commit e3b6eaf

Please sign in to comment.