Skip to content
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

Merged
merged 5 commits into from
Nov 25, 2023
Merged

Restore the codecov comments on PRs #1173

merged 5 commits into from
Nov 25, 2023

Conversation

mwouts
Copy link
Owner

@mwouts mwouts commented Nov 24, 2023

The objective is to close #1171 and #1172

Copy link

codecov bot commented Nov 25, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (923798e) 96.69% compared to head (b05b0b8) 97.59%.

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     
Flag Coverage Δ
external 75.23% <ø> (ø)
functional 88.54% <ø> (ø)
integration 76.84% <ø> (ø)
unit 66.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mwouts mwouts merged commit e1c6d13 into main Nov 25, 2023
29 checks passed
@mwouts mwouts deleted the codecov_comment branch November 25, 2023 16:48
@mwouts mwouts mentioned this pull request Nov 25, 2023
Copy link
Contributor

@LecrisUT LecrisUT left a 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
Copy link
Contributor

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"]
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Restore the codecov comments on the PRs
2 participants