Skip to content

Commit

Permalink
[-] remove race check from TestProcessorErr
Browse files Browse the repository at this point in the history
  • Loading branch information
bnkamalesh committed Nov 4, 2024
1 parent 8860f4f commit ac4eaba
Showing 1 changed file with 35 additions and 31 deletions.
66 changes: 35 additions & 31 deletions .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,36 +4,40 @@
name: Go

on:
push:
branches: ["main"]
pull_request:
branches: ["main"]
push:
branches: ["main"]
pull_request:
branches: ["main"]

jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Set up Go
uses: actions/setup-go@v4
with:
go-version: "1.23"

- name: Build
run: go build -v ./...

- name: Tests
run: |
go install github.com/mattn/goveralls@latest
go test -race -covermode atomic -coverprofile=covprofile ./...
- name: Send coverage
uses: shogo82148/actions-goveralls@v1
with:
path-to-profile: covprofile

- name: golangci-lint
uses: golangci/golangci-lint-action@v6
with:
version: v1.60
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Set up Go
uses: actions/setup-go@v4
with:
go-version: "1.23"

- name: Build
run: go build -v ./...

- name: Tests
run: |
go install github.com/mattn/goveralls@latest
go test -race -run "TestNibbler|TestSanitizeValidate"
# race check is ignored for this because of how the test (TestProcessorErr) itself relies
# on a panic, when pushing to a closed channel
go test -covermode atomic -coverprofile=covprofile ./...
- name: Send coverage
uses: shogo82148/actions-goveralls@v1
with:
path-to-profile: covprofile

- name: golangci-lint
uses: golangci/golangci-lint-action@v6
with:
version: v1.60

0 comments on commit ac4eaba

Please sign in to comment.