Skip to content

Remove PyPI codecov - #163

Merged
bsipocz merged 3 commits into
astropy:mainfrom
ConorMacBride:remove-codecov
Apr 13, 2023
Merged

bsipocz merged 3 commits into
astropy:mainfrom
ConorMacBride:remove-codecov

Conversation

@ConorMacBride

Copy link
Copy Markdown
Member

Remove the codecov PyPI package from the dependencies. The workflow is already using the codecov uploader action, so this should not be needed, unless coverage needs to uploaded from elsewhere?

@bsipocz

bsipocz commented Apr 13, 2023

Copy link
Copy Markdown
Member

Thanks! I'm going ahead and merge this, I'm afraid the failing job is likely highlights some incompetency with sphinx 6.2 :(

@bsipocz

bsipocz commented Apr 13, 2023

Copy link
Copy Markdown
Member

I've pushed more CI related changes to this PR, IMO monthly crons are useless, it doesn't hurt to have them weekly instead. And for low-traffic repos like this, canceling jobs on failure is more annoying than useful, too.

@codecov

codecov Bot commented Apr 13, 2023

Copy link
Copy Markdown

Codecov Report

Merging #163 (8ba873d) into main (8b2dbcf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #163   +/-   ##
=======================================
  Coverage   89.83%   89.83%           
=======================================
  Files          27       27           
  Lines        1141     1141           
=======================================
  Hits         1025     1025           
  Misses        116      116           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bsipocz

bsipocz commented Apr 13, 2023

Copy link
Copy Markdown
Member

See #164 as a follow-up issue for the sphinx dev problem, otherwise this PR fixes problems, so I go ahead and merge it now.

@bsipocz
bsipocz merged commit d29f447 into astropy:main Apr 13, 2023
@ConorMacBride
ConorMacBride deleted the remove-codecov branch April 13, 2023 23:04
@pllim

pllim commented Apr 14, 2023

Copy link
Copy Markdown
Member

Thanks, all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants