-
Notifications
You must be signed in to change notification settings - Fork 392
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
Restore the codecov comments on PRs #1173
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1173 +/- ##
==========================================
+ Coverage 96.69% 97.59% +0.89%
==========================================
Files 26 26
Lines 4357 4357
==========================================
+ Hits 4213 4252 +39
+ Misses 144 105 -39
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Because otherwise the commit always get a red cross
c7f83ca
to
b05b0b8
Compare
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.
Two quick comments
|
||
- name: Upload coverage | ||
uses: codecov/codecov-action@v3 | ||
with: | ||
fail_ci_if_error: true |
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.
fail_ci_if_error
has no effect here because this step is not triggered if the test step failed. You can add if: always()
if you do want it to upload failed code coverage.
matrix: | ||
python-version: [ "3.8", "3.9", "3.10", "3.11" ] | ||
markdown-it-py-version: ["~=2.0"] | ||
python-version: [ "3.8", "3.9", "3.10", "3.11", "3.12-dev"] |
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.
Python 3.12 has been officially out. Should be included soon.
The objective is to close #1171 and #1172