Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing race conditions #30

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fixing race conditions #30

wants to merge 1 commit into from

Conversation

rwrz
Copy link

@rwrz rwrz commented Dec 29, 2022

I'm using this library with -race and Go warned me about some possible race conditions.
This commit fixes it.

All tests are still working. 馃

@coveralls
Copy link

Coverage Status

Coverage: 91.173% (+0.1%) from 91.052% when pulling 68aad8b on rwrz:fix-race into 5f246de on nats-io:master.

@derekcollison
Copy link
Member

Need to fix one failure on CI failure.

https://app.travis-ci.com/github/nats-io/graft/jobs/592230016

@rwrz
Copy link
Author

rwrz commented Dec 31, 2022

Seems to be just a download error. Can you retry?

@derekcollison
Copy link
Member

derekcollison commented Jan 1, 2023

go 1.16 is complaining about the following.

github.com/nats-io/nats-server/v2@v2.9.10/server/events.go:31:2: //go:build comment without // +build comment

I might suggest bumping the Go version in the .travis file to go 1.18 and go 1.19 and see if it clears.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants