Skip to content

Issue 0031/remove dot coverage#32

Merged
nigelmegitt merged 4 commits into
mainfrom
issue-0031/remove-dot-coverage
Jun 19, 2026
Merged

Issue 0031/remove dot coverage#32
nigelmegitt merged 4 commits into
mainfrom
issue-0031/remove-dot-coverage

Conversation

@nigelmegitt

Copy link
Copy Markdown
Collaborator

Closes #31

PR should make the CI workflow succeeds by working around coveragepy/coveragepy#2197 as suggested by @ewjoachim.

@nigelmegitt nigelmegitt requested review from a team and rolandtreiber June 19, 2026 10:19
@nigelmegitt nigelmegitt added bug Something isn't working CI/CD Stuff to do with continuous integration or continuous deployment labels Jun 19, 2026
Signed-off-by: Nigel Megitt <nigel.megitt@bbc.co.uk>
Signed-off-by: Nigel Megitt <nigel.megitt@bbc.co.uk>
@nigelmegitt nigelmegitt force-pushed the issue-0031/remove-dot-coverage branch from 44f1533 to 134f4ca Compare June 19, 2026 10:20
@nigelmegitt

Copy link
Copy Markdown
Collaborator Author

The expectation was that this should pass the CI workflow, but it's failing for the same reason as before the .coverage.license and .coverage files were removed.

It's unclear why - one thing to investigate is if the reason is that it's comparing the coverage on main which still has this issue, or if it's something to do with what is in the upstream/python-coverage-comment-action-data branch.

Re-running the action with debug logging enabled is probably a useful first step in investigation.

Comment thread docs/.coverage Outdated
Signed-off-by: Nigel Megitt <nigel.megitt@bbc.co.uk>
Signed-off-by: Nigel Megitt <nigel.megitt@bbc.co.uk>
@github-actions

Copy link
Copy Markdown

Coverage report

This PR does not seem to contain any modification to coverable code.

@nigelmegitt nigelmegitt merged commit ce7e706 into main Jun 19, 2026
5 checks passed
@nigelmegitt nigelmegitt deleted the issue-0031/remove-dot-coverage branch June 19, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working CI/CD Stuff to do with continuous integration or continuous deployment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove and ignore .coverage*

2 participants