-
Notifications
You must be signed in to change notification settings - Fork 71
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: fix codecov #2650
ci: fix codecov #2650
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2650 +/- ##
==========================================
+ Coverage 89.61% 89.64% +0.02%
==========================================
Files 360 361 +1
Lines 46209 46252 +43
Branches 6986 6992 +6
==========================================
+ Hits 41410 41462 +52
+ Misses 3724 3719 -5
+ Partials 1075 1071 -4 ☔ View full report in Codecov by Sentry. |
@superlopuh, is this working (i.e, can I merge it?) |
No, please see discussion on linked issue in codecov, this PR is useful for me to keep debugging the problem. It seems like merging it would still be a good move, though, I'll hit the button and maybe open more PRs to get to the bottom of the issue. |
As recommended here:
codecov/feedback#388
This PR makes it such that checkout fetches the two prior commits, to show the diff.