-
Notifications
You must be signed in to change notification settings - Fork 337
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
ci: use codecov #2559
ci: use codecov #2559
Conversation
eb1a734
to
a7effa5
Compare
Codecov Report
@@ Coverage Diff @@
## master #2559 +/- ##
=========================================
Coverage ? 63.83%
=========================================
Files ? 232
Lines ? 25734
Branches ? 0
=========================================
Hits ? 16428
Misses ? 7823
Partials ? 1483 Continue to review full report at Codecov.
|
0d44cca
to
345c5ef
Compare
345c5ef
to
cd16384
Compare
Kudos, SonarCloud Quality Gate passed! |
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.
Nice!
- name: Coveralls | ||
- name: Test with race detector | ||
if: matrix.os != 'ubuntu-latest' | ||
run: make test-race |
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.
@acud you will also execute race-tests on Windows, I know that they were disabled for Windows, maybe its time to enable them again, don't know
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.
yep i enabled them on purpose 👍
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)