-
-
Notifications
You must be signed in to change notification settings - Fork 57
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Misc cleanups #209
Misc cleanups #209
Conversation
gwynne
commented
May 27, 2023
•
edited
Loading
edited
- Cleanup CI a little
- Update README
- Move CONTRIBUTING.md to .github
- Make tests more reliable
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #209 +/- ##
=======================================
Coverage 55.58% 55.58%
=======================================
Files 11 11
Lines 412 412
=======================================
Hits 229 229
Misses 183 183 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems like the CI is failing because there are some new Thread Sanitizer stuff on Swift 5.8+.
I'd rather a happy Thread Sanitizer than not, but that is unrelated to what changes this PR has made.
The TSan failures are solved in https://gitlab.com/swift-server-community/RediStack/-/merge_requests/196 and https://gitlab.com/swift-server-community/RediStack/-/merge_requests/197. |