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

chore: update codecov-action to v4 #704

Merged
merged 1 commit into from
Apr 20, 2024

Conversation

vil02
Copy link
Member

@vil02 vil02 commented Feb 2, 2024

Same as TheAlgorithms/Rust#670 - it contains the description of the problem.

Before merging, please add the token from https://app.codecov.io/gh/TheAlgorithms/Go/settings as CODECOV_TOKEN into the repo secrets. Token was added.

@vil02 vil02 marked this pull request as ready for review February 2, 2024 18:27
@codecov-commenter
Copy link

codecov-commenter commented Feb 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.25%. Comparing base (cbaed23) to head (e61420b).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #704      +/-   ##
==========================================
+ Coverage   86.84%   87.25%   +0.40%     
==========================================
  Files         198      199       +1     
  Lines        6585     5274    -1311     
==========================================
- Hits         5719     4602    -1117     
+ Misses        727      533     -194     
  Partials      139      139              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

github-actions bot commented Mar 4, 2024

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added stale and removed stale labels Mar 4, 2024
Copy link

github-actions bot commented Apr 4, 2024

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the stale label Apr 4, 2024
@vil02
Copy link
Member Author

vil02 commented Apr 4, 2024

Please 'dont-close' or merge.

@vil02 vil02 force-pushed the update_codecov_action_to_v4 branch from ac0fa84 to 516bf62 Compare April 4, 2024 14:32
@vil02 vil02 force-pushed the update_codecov_action_to_v4 branch from 516bf62 to e61420b Compare April 17, 2024 16:32
Copy link
Member

@siriak siriak left a comment

Choose a reason for hiding this comment

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

I have added the token. Is it ready to be merged?

@vil02 vil02 changed the title [DO NOT MERGE before adding the token] chore: update codecov-action to v4 chore: update codecov-action to v4 Apr 19, 2024
@vil02
Copy link
Member Author

vil02 commented Apr 19, 2024

Is it ready to be merged?

From my side: yes.

@siriak siriak merged commit 0d0b97a into TheAlgorithms:master Apr 20, 2024
5 checks passed
sozelfist pushed a commit to sozelfist/Go that referenced this pull request Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants