Skip to content

Conversation

@cpelley
Copy link
Contributor

@cpelley cpelley commented Dec 18, 2025

Fix and simplified the recently merged coverage report gen/publish via actions.
I think the bug slipped in through the many subsequent merges of master into the dev branch before final merging.

Let's get this in ASAP before master changes 😋

While doing this, I implemented improved caching for this particular job so that I could quickly iterate with testing the action (no longer re-runs pytest to generate a new report if the relevant files haven't changed).

@github-actions
Copy link

The test coverage has decreased from 91% to 91% (commit SHA: 25942b4).
Please review test coverage here

@metoppv metoppv deleted a comment from github-actions bot Dec 19, 2025
@metoppv metoppv deleted a comment from github-actions bot Dec 19, 2025
@github-actions
Copy link

The test coverage has decreased from 91% to 91% (commit SHA: 998d371).
Please review test coverage here

@cpelley cpelley marked this pull request as ready for review December 19, 2025 08:51
@cpelley cpelley changed the title debug change BUG: gh actions coverage report Dec 19, 2025
@cpelley cpelley self-assigned this Dec 19, 2025
Copy link
Contributor

@gavinevans gavinevans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @cpelley 👍

This seems to have fixed the issue.

@cpelley cpelley merged commit 69e6d9b into master Dec 19, 2025
7 checks passed
@cpelley cpelley deleted the test_coverage_fix branch December 19, 2025 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants