-
Notifications
You must be signed in to change notification settings - Fork 526
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
Unpin Codecov / Update coverage #3303
Conversation
It looks like the Codecov uploads originating from Jenkins are what's messing with the tokenless upload. I think the Jenkins job is mistakenly uploading coverage reports to the main branch. Tokenless currently requires that the branch name specified when uploading follow a specific format, more info here. What's happening is that the Jenkins job is uploading to main, which means that Codecov pins the commit SHA being uploaded to being on main, and ineligible for tokenless uploads. Later, when the codecov-action tries to upload to that same commit sha, Codecov sees that it already exists and is on the Also it seems like the Jenkins job is using a very old version of the Codecov uploader, so I would suggest upgrading it to use the CLI, more info here. |
I attempted to install $ codecovcli --version
Traceback (most recent call last):
File "/packages/python/codecov-cli/0.7.2/bin/codecovcli", line 5, in <module>
from codecov_cli.main import run
File "/packages/python/codecov-cli/0.7.2/lib/python3.8/site-packages/codecov_cli/main.py", line 14, in <module>
from codecov_cli.commands.process_test_results import process_test_results
File "/packages/python/codecov-cli/0.7.2/lib/python3.8/site-packages/codecov_cli/commands/process_test_results.py", line 8, in <module>
from test_results_parser import (
ModuleNotFoundError: No module named 'test_results_parser' EDIT: One of the dependencies didn't install. My goodness. Okay, continuing to work on this... |
Fixes NA
Summary/Motivation:
We've been having lots of issues with codecov lately. This is an attempt to fix that.
Changes proposed in this PR:
codecov.yml
formatLegal Acknowledgement
By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution: