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

[Codecov] Install codecov-cli instead of the Codecov uploader #615

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

amenasria
Copy link
Member

@amenasria amenasria commented Jun 24, 2024

Why we're replacing the legacy uploader by the codecov-cli binary:

TL; DR: the 'uploader' has been deprecated since 2023 (see screen below). The right way to upload coverage files to Codecov now is to use the codecov-cli binary. I kept it before as it was the legacy way we were using Codecov, but we're now catching up with upstream because:

  • We're missing interesting features:
    • Have the UI build URLs redirecting directly to our gitlab jobs.
    • Be able to get Codecov information right after you pushed to remote (this will be needed for later as it might be a mandatory check in the CI).
  • Security
  • Maintenance / ease of setup (doc is outdated sometimes)

@amenasria amenasria requested review from a team as code owners June 24, 2024 16:23
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