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

Tests: Generate an HTML coverage report when running tests locally. #525

Merged
merged 1 commit into from
Feb 21, 2023

Conversation

iandunn
Copy link
Member

@iandunn iandunn commented Feb 15, 2023

This makes it convenient to check coverage locally, instead of having to go to Coveralls.

This does part of the documentation for #467 , but only the local part. Docs will still need to be added for Coveralls when #364 is done.

This makes it convenient to check coverage locally, instead of having to go to Coveralls.
@iandunn iandunn marked this pull request as ready for review February 15, 2023 21:40
@iandunn iandunn requested review from kasparsd and dd32 February 15, 2023 21:40
@iandunn iandunn self-assigned this Feb 15, 2023
@iandunn
Copy link
Member Author

iandunn commented Feb 15, 2023

I'm having trouble getting the tests to run in Docker (even on master). If anyone has it working and can test that it still works with this PR, that'd be helpful. The tests passed in the GH action, though, so it should be fine.

@iandunn
Copy link
Member Author

iandunn commented Feb 21, 2023

I'm gonna go ahead and merge this, but LMK if anyone sees any problems in the future.

@iandunn iandunn merged commit 8df5940 into master Feb 21, 2023
@jeffpaul jeffpaul added this to the 0.8.0 milestone Feb 22, 2023
@jeffpaul jeffpaul deleted the local-coverage branch March 2, 2023 15:37
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.

2 participants