Skip to content

CI: remove the current coverage build, run coverage on multiple builds instead #39758

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

Closed
fangchenli opened this issue Feb 11, 2021 · 1 comment · Fixed by #40394 or #39666
Closed

CI: remove the current coverage build, run coverage on multiple builds instead #39758

fangchenli opened this issue Feb 11, 2021 · 1 comment · Fixed by #40394 or #39666
Labels
CI Continuous Integration Needs Discussion Requires discussion from core team before further action
Milestone

Comments

@fangchenli
Copy link
Member

Currently, we upload one coverage report from actions-37-cov. It takes the longest. But Codecov automatically merges multiple reports. See details https://docs.codecov.io/docs/merging-reports. So we could run coverage on multiple shorter builds.

We need to select a set of builds (create new ones if needed), add -cov to the name of their environment files (suggested by @jreback ), update their CI configs to upload reports to Codecov, and remove the actions-37-cov build if it's no longer required.

@fangchenli fangchenli added Bug Needs Triage Issue that has not been reviewed by a pandas team member CI Continuous Integration Needs Discussion Requires discussion from core team before further action and removed Bug Needs Triage Issue that has not been reviewed by a pandas team member labels Feb 11, 2021
@lithomas1 lithomas1 self-assigned this Mar 9, 2021
@lithomas1 lithomas1 removed their assignment Mar 12, 2021
@jreback jreback added this to the 1.3 milestone Mar 14, 2021
@MarcoGorelli
Copy link
Member

Reopening as #40394 didn't remove actions-37-cov, but i think i that can be done now if there's no issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration Needs Discussion Requires discussion from core team before further action
Projects
None yet
4 participants