Skip to content

Fix coveralls reporting#63

Merged
dwhswenson merged 2 commits intomasterfrom
fix_coveralls_parallel
Nov 12, 2019
Merged

Fix coveralls reporting#63
dwhswenson merged 2 commits intomasterfrom
fix_coveralls_parallel

Conversation

@dwhswenson
Copy link
Owner

Coveralls is reporting each build separately, instead of combining them. There are multiple instances of each file when looking at coverage data. This is trying to fix that.

@dwhswenson dwhswenson changed the title [WIP] Fix coveralls reporting Fix coveralls reporting Oct 21, 2019
@dwhswenson
Copy link
Owner Author

Apparently I completely forgot that I hadn't merged this PR. It seems like it might have fixed the problem, so I'll go ahead and merge, which should mean that other PRs can actually get the full coverage they deserve!

@dwhswenson dwhswenson merged commit 09289cd into master Nov 12, 2019
@dwhswenson dwhswenson deleted the fix_coveralls_parallel branch November 12, 2019 12:07
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.

1 participant