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

Update code coverage tool to use 'Code Coverage Summary' instead of Codecov #30

Merged
merged 11 commits into from
Oct 23, 2024
Merged

Update code coverage tool to use 'Code Coverage Summary' instead of Codecov #30

merged 11 commits into from
Oct 23, 2024

Conversation

ruokun-niu
Copy link
Contributor

@ruokun-niu ruokun-niu commented Oct 9, 2024

Description

We were using Codecov as code coverage report platform earlier. However, when we erased the history of the repository, Codecov stopped working as it was pointing at an old commit (this commit does not belong to any repository).

I have attempted to configure Codecov to point to the correct commit, but I was not able to do so. After investigating further, seems like the best approach right now is to just ditch Codecov and use an alternative. Specifically, I found a popular tool called 'Code Coverage Summary' and it is much easier to use.

Why we should use Code Coverage Summary:

Some tradeoffs from not using Codecov:

  • Code coverage summary does not have a dashboard (we still have a pretty good report)
  • Difficult to compare the code coverage changes between the current run and the previous ones

Overall, I think this code coverage report tool is just a lot easier to use and maintain and less error prone.

Type of change

  • This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Drasi (issue link optional).

Fixes: #issue_number

@ruokun-niu ruokun-niu changed the title DRAFT: Updated code coverage tool Update code coverage tool to use 'Code Coverage Summary' instead of Codecov Oct 9, 2024
@ruokun-niu ruokun-niu marked this pull request as ready for review October 9, 2024 21:17
@ruokun-niu ruokun-niu merged commit 349a344 into drasi-project:main Oct 23, 2024
9 checks passed
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.

3 participants