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 coveralls workflow to fetch commit history #186

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

saiponnada
Copy link
Contributor

Problem

In the context of GitHub Actions, the actions/checkout action supports the fetch-depth parameter to control how much history is fetched when checking out the repository. Default value of fetch-depth is 1 which gives access to only latest commit.

Solution

But, Coveralls needs to access the commit history to generate accurate coverage reports and associate them with the correct commits. If only the latest commit is fetched (fetch-depth: 1), Coveralls might not be able to find the necessary commit information, leading to errors like "Not a valid object name." Hence setting to 0 to fetch entire commit history.

@saiponnada saiponnada self-assigned this Oct 22, 2024
@saiponnada saiponnada merged commit c4eaee3 into master Oct 22, 2024
1 check passed
@saiponnada saiponnada deleted the fix-coveralls branch October 22, 2024 21:31
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