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

🚑🧪 Set the Codecov token directly in GHA #12516

Merged

Conversation

webknjaz
Copy link
Member

It's necessary since it seems that the currently used Codecov uploader doesn't read the token from the config sometimes.

This is a follow-up for #12508 which wasn't enough.

It's necessary since it seems that the currently used Codecov uploader
doesn't read the token from config sometimes.

This is a follow-up for pytest-dev#12508 which wasn't enough.
@webknjaz webknjaz added backport 8.2.x skip news used on prs to opt out of the changelog requirement labels Jun 21, 2024
@webknjaz webknjaz enabled auto-merge June 21, 2024 21:01
@webknjaz
Copy link
Member Author

@RonnyPfannschmidt @The-Compiler approve plz?

@webknjaz webknjaz merged commit f75e3fe into pytest-dev:main Jun 22, 2024
29 checks passed
Copy link

patchback bot commented Jun 22, 2024

Backport to 8.2.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/8.2.x/f75e3fe63f8d22422393888bd5b9094cab35f522/pr-12516

Backported as #12520

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Jun 22, 2024
webknjaz added a commit that referenced this pull request Jun 22, 2024
…75e3fe63f8d22422393888bd5b9094cab35f522/pr-12516

[PR #12516/f75e3fe6 backport][8.2.x] 🚑🧪 Set the Codecov token directly in GHA
@webknjaz
Copy link
Member Author

FTR, here I'm trying to get this regression fixed in the Codecov CLI incarnation of the uploader they use for v4 of their official action nowadays: codecov/codecov-cli#464.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news used on prs to opt out of the changelog requirement type: infrastructure improvement to development/releases/CI structure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants