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

Downgrade codecov #1686

Merged

Conversation

Polber
Copy link
Contributor

@Polber Polber commented Jun 25, 2024

Temporary downgrade to mitigate issue with not finding token on forked PR runs.

Tracked in codecov/codecov-action#1487

@Polber Polber requested a review from a team as a code owner June 25, 2024 15:33
@Polber Polber requested review from Deep1998 and shreyakhajanchi and removed request for a team June 25, 2024 15:33
@Polber Polber force-pushed the jkinard/test-codecov branch 3 times, most recently from 647b602 to 0672740 Compare June 28, 2024 19:05
@Polber Polber closed this Jul 3, 2024
@Polber Polber reopened this Jul 3, 2024
Copy link

codecov bot commented Jul 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.26%. Comparing base (0bcf179) to head (5663044).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1686   +/-   ##
=========================================
  Coverage     41.25%   41.26%           
- Complexity     3240     3242    +2     
=========================================
  Files           771      771           
  Lines         45127    45127           
  Branches       4819     4819           
=========================================
+ Hits          18619    18622    +3     
+ Misses        24940    24938    -2     
+ Partials       1568     1567    -1     
Components Coverage Δ
spanner-templates 61.27% <ø> (+0.02%) ⬆️
spanner-import-export 64.42% <ø> (+0.06%) ⬆️
spanner-live-forward-migration 74.14% <ø> (ø)
spanner-live-reverse-replication 50.56% <ø> (ø)
spanner-bulk-migration 80.63% <ø> (ø)

see 2 files with indirect coverage changes

Signed-off-by: Jeffrey Kinard <jeff@thekinards.com>
@Polber Polber changed the title [DO NOT MERGE] test codecov Downgrade codecov Jul 3, 2024
@Polber Polber requested a review from damccorm July 3, 2024 15:28
Copy link
Contributor

@damccorm damccorm left a comment

Choose a reason for hiding this comment

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

Thanks

@damccorm damccorm added the Google LGTM Approval of a pull request to be merged into the repository label Jul 3, 2024
@copybara-service copybara-service bot merged commit 0fef11c into GoogleCloudPlatform:main Jul 3, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Google LGTM Approval of a pull request to be merged into the repository size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants