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 the workflow to use Codecov as the coverage tool #40

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

ruokun-niu
Copy link
Contributor

Description

We previously switched from using Codecov to another code coverage tool as Codecov stopped working when we erased the commit history. There were also some issues with their tokenless uploads, which prevented code coverage results from being uploaded from a fork branch.

I have filed an issue in Codecov's repository. One of the maintainers reached out and diagnosed the issue with me. I was able to do a successful upload in a recent test

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

Copy link

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 92b6e31. ± Comparison against base commit b391fce.

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.

1 participant