-
-
Notifications
You must be signed in to change notification settings - Fork 167
ci: integrate Sentry Prevent #911
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #911 +/- ##
=======================================
Coverage 73.46% 73.46%
=======================================
Files 64 64
Lines 7518 7518
=======================================
Hits 5523 5523
Misses 1995 1995 |
816a50c
to
a3ffc5f
Compare
It seems that the JUnit format output by nextest is not compatible with Prevent/Codecov. Update: actually it's supported, there's just something unexpected in the UI where it wasn't obvious where I would see the test runs. |
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.
lgtm, but check comments before merge
|
||
- name: Run cargo test | ||
run: cargo test --workspace --all-features --all-targets | ||
- uses: taiki-e/install-action@nextest |
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.
You may wish to consider pinning GitHub Actions by hash
#skip-changelog