Skip to content

Conversation

@vil02
Copy link
Member

@vil02 vil02 commented Apr 6, 2024

With this change, one can generate a coverage report by calling any command like

mvn test

A html, csv and xml reports are generated (this is useful for local development in my opinion).

The xml is uploaded to codecov.

Please note that this change still uses codecov/codecov-action in v3 - the update requires adding a token, so it will be done in a separate PR.

  • 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
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (master@c53f178). Click here to learn what that means.

Additional details and impacted files
@@ Coverage Diff @@ ## master #5098 +/- ## ========================================= Coverage ? 37.13% Complexity ? 2331 ========================================= Files ? 517 Lines ? 15645 Branches ? 2984 ========================================= Hits ? 5809 Misses ? 9548 Partials ? 288 

☔ 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 08:21
@vil02
Copy link
Member Author

vil02 commented Apr 6, 2024

@BamaCharanChhandogi please have a look.

@BamaCharanChhandogi BamaCharanChhandogi merged commit ab094ef into TheAlgorithms:master Apr 6, 2024
@vil02 vil02 deleted the add_coverage_report branch April 6, 2024 15:24
vil02 added a commit to vil02/Java that referenced this pull request Apr 6, 2024
vil02 added a commit to vil02/Java that referenced this pull request Apr 6, 2024
vil02 added a commit to vil02/Java that referenced this pull request Apr 8, 2024
BamaCharanChhandogi pushed a commit that referenced this pull request Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants