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

Remove PyPI codecov #163

Merged
merged 3 commits into from Apr 13, 2023
Merged

Remove PyPI codecov #163

merged 3 commits into from Apr 13, 2023

Conversation

ConorMacBride
Copy link
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
Copy link
Member

bsipocz commented Apr 13, 2023

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

@bsipocz
Copy link
Member

bsipocz commented Apr 13, 2023

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
Copy link

codecov bot commented Apr 13, 2023

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
Copy link
Member

bsipocz commented Apr 13, 2023

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
16 of 19 checks passed
@ConorMacBride ConorMacBride deleted the remove-codecov branch April 13, 2023 23:04
@pllim
Copy link
Member

pllim commented Apr 14, 2023

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.

None yet

3 participants