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.

Sign up for GitHub

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

Jump to bottom

fix: coverage action fails to upload #197

Closed
wants to merge 1 commit into from
Closed

fix: coverage action fails to upload #197

wants to merge 1 commit into from

Conversation

jvallesm
Copy link
Collaborator

@jvallesm jvallesm commented Apr 11, 2024 •
edited
Loading

Because

  • Coverage upload behaviour is broken. Action point here is adding tokens to our backend repos, but upgrading to codecov@v4 has been reported to address the issue

This commit

  • Updates the coverage GHA to codecov v4.

Part of INS-4122

@jvallesm jvallesm self-assigned this Apr 11, 2024
@jvallesm
Copy link
Collaborator Author

Doesn't solve the issue as mentioned in this comment

@jvallesm jvallesm closed this Apr 11, 2024
@linear Linear
Copy link

linear bot commented Apr 11, 2024

INS-4122 List and add token to 'codecov' workflow

@donch1989 donch1989 deleted the jvalles/fix-codecov-upload branch June 5, 2024 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@donch1989 donch1989 Awaiting requested review from donch1989 donch1989 will be requested when the pull request is marked ready for review donch1989 is a code owner

@xiaofei-du xiaofei-du Awaiting requested review from xiaofei-du xiaofei-du will be requested when the pull request is marked ready for review xiaofei-du is a code owner

@pinglin pinglin Awaiting requested review from pinglin pinglin will be requested when the pull request is marked ready for review pinglin is a code owner

Assignees

@jvallesm jvallesm

Labels
instill core
Projects
No open projects
🔮 Instill Core
Status: 👋 Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jvallesm @droplet-bot