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 Upload Reports fix #2344

Merged
merged 2 commits into from Jun 7, 2021
Merged

Codecov Upload Reports fix #2344

merged 2 commits into from Jun 7, 2021

Conversation

ParthivNaresh
Copy link
Contributor

@ParthivNaresh ParthivNaresh commented Jun 7, 2021

Fixes issue with Codecov where reports were not being uploaded properly. From now on the CODECOV_TOKEN will be passed to authorize the upload.

@@ -83,4 +83,5 @@ jobs:
name: Upload coverage to Codecov
uses: codecov/codecov-action@v1
with:
token: ${{ secrets.CODECOV_TOKEN }}
Copy link
Contributor

Choose a reason for hiding this comment

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

So something changed with the codecov/codecov-action@v1 action which now requires this token, whereas previously we didn't have to provide it?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Correct me if I'm wrong, but the @v1 stuff refers to which version of codecov-action we're checking out, right? I think, if you're interested in previous versions of codecov action, you can change the 1 to a 2 or also specify the specific version of codecov-action you want to try to see whether the latest requires the token or not. Still, very weird that it doesn't correspond to a recent release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not so sure that it's an issue with the codecov action, it doesn't seem to have released anything recently. It might be related to how secrets are accessed by GitHub Actions?

Copy link
Contributor

@dsherry dsherry left a comment

Choose a reason for hiding this comment

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

LGTM on green tests! 🚢 🙏

@codecov
Copy link

codecov bot commented Jun 7, 2021

Codecov Report

Merging #2344 (64ed615) into main (482bab1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #2344   +/-   ##
=====================================
  Coverage   99.9%   99.9%           
=====================================
  Files        281     281           
  Lines      24607   24607           
=====================================
  Hits       24578   24578           
  Misses        29      29           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 482bab1...64ed615. Read the comment docs.

Copy link
Contributor

@freddyaboulton freddyaboulton left a comment

Choose a reason for hiding this comment

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

Thank you @ParthivNaresh !

@ParthivNaresh ParthivNaresh merged commit 432f876 into main Jun 7, 2021
@ParthivNaresh ParthivNaresh deleted the Codecov-Upload-Fix branch June 7, 2021 20:45
This was referenced Jun 9, 2021
@chukarsten chukarsten mentioned this pull request Jun 22, 2021
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.

None yet

4 participants