Skip to content

chore: update codecov/codecov-action to v4 - #5100

Merged
siriak merged 3 commits into
TheAlgorithms:masterfrom
vil02:update_codecov_action_to_v4
Apr 6, 2024
Merged

chore: update codecov/codecov-action to v4#5100
siriak merged 3 commits into
TheAlgorithms:masterfrom
vil02:update_codecov_action_to_v4

Conversation

@vil02

@vil02vil02 commented Apr 6, 2024

Copy link
Copy Markdown
Member

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/Java/settings as CODECOV_TOKEN into the repo secrets, including dependabot secrets.Token is added.

@siriak - your help will be needed here.

Continuation of #5098.

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • All new code is formatted with clang-format -i --style=file path/to/your/file.java

@vil02vil02 changed the title chore: update codecov/codecov-action to v4[DO NOT MERGE before adding the token] chore: update codecov/codecov-action to v4Apr 6, 2024
@vil02
vil02force-pushed the update_codecov_action_to_v4 branch from e7226e5 to 351f0eeCompareApril 6, 2024 15:43
@vil02
vil02force-pushed the update_codecov_action_to_v4 branch from 351f0ee to 0d9ef27CompareApril 6, 2024 15:45
@codecov-commenter

codecov-commenter commented Apr 6, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 37.11%. Comparing base (ab094ef) to head (ee1ad00).

Additional details and impacted files
@@ Coverage Diff @@## master #5100 +/- ##
============================================
- Coverage 37.13% 37.11% -0.02% + Complexity 2331 2330 -1 
============================================
Files 517 517 Lines 15645 15645 Branches 2984 2984 ============================================
- Hits 5809 5807 -2 - Misses 9548 9549 +1 - Partials 288 289 +1 

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

@vil02
vil02 marked this pull request as ready for review April 6, 2024 15:50
@vil02
vil02 requested a review from siriakApril 6, 2024 15:51
@vil02
vil02 marked this pull request as draft April 6, 2024 16:01
@vil02
vil02force-pushed the update_codecov_action_to_v4 branch from 3d7be11 to df310e8CompareApril 6, 2024 16:19
@vil02
vil02force-pushed the update_codecov_action_to_v4 branch from df310e8 to 25453f8CompareApril 6, 2024 16:43
@vil02
vil02 marked this pull request as ready for review April 6, 2024 16:45

@siriaksiriak left a comment

Copy link
Copy Markdown
Member

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 to actions and dependabot sections. Is it ready to be merged or you want to test something?

@vil02

vil02 commented Apr 6, 2024

Copy link
Copy Markdown
MemberAuthor

I have added the token to actions and dependabot sections. Is it ready to be merged or you want to test something?

Thanks. It is ready to be merged.

@vil02vil02 changed the title [DO NOT MERGE before adding the token] chore: update codecov/codecov-action to v4chore: update codecov/codecov-action to v4Apr 6, 2024
@vil02
vil02 enabled auto-merge (squash) April 6, 2024 18:28
@siriak
siriak disabled auto-merge April 6, 2024 18:50
@siriak
siriak merged commit 295ac4b into TheAlgorithms:masterApr 6, 2024
@vil02
vil02 deleted the update_codecov_action_to_v4 branch April 6, 2024 18:50
Sign up for freeto 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.

3 participants

@vil02@codecov-commenter@siriak