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

ci: Split codecov uploads into a separate workflow #3459

Merged
merged 1 commit into from
May 30, 2024

Conversation

jplatte
Copy link
Collaborator

@jplatte jplatte commented May 23, 2024

… which runs in the context of the main repo even for PRs from forks, and can be retried individually without rerunning coverage collection.

I saw you're still suffering from codecov's problems, so I decided to take the time required to look up the most recent suggested workaround (most up-to-date upstream issue discussing this stuff is codecov/feedback#301 and contains links to the workflows I copied here).

Hope it works. If not, maybe you can get it to work. Can't promise that I'll put more time into this.

… which runs in the context of the main repo even for PRs, and can be
retried individually without rerunning coverage collection.
@jplatte jplatte requested a review from a team as a code owner May 23, 2024 18:45
@jplatte jplatte requested review from bnjbvr and removed request for a team May 23, 2024 18:45
@jplatte
Copy link
Collaborator Author

jplatte commented May 23, 2024

Hm, new workflow file doesn't trigger, prob. will only start doing that after it's part of main?

@bnjbvr
Copy link
Member

bnjbvr commented May 28, 2024

I'm curious as to why it didn't trigger in this branch; shouldn't it? I'm pretty sure I've seen in the past that new CI steps would run in the PR that introduced them…

@bnjbvr bnjbvr requested review from a team and Hywan and removed request for bnjbvr and a team May 28, 2024 08:52
@bnjbvr bnjbvr changed the title Split codecov uploads into a separate workflow ci: Split codecov uploads into a separate workflow May 30, 2024
Copy link
Member

@bnjbvr bnjbvr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's try it out!

@bnjbvr bnjbvr enabled auto-merge (rebase) May 30, 2024 08:35
@bnjbvr bnjbvr merged commit 40dc706 into main May 30, 2024
36 checks passed
@bnjbvr bnjbvr deleted the jplatte/codecov-upload branch May 30, 2024 11:53
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.

2 participants