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

Add hatch command for html coverage report #2721

Merged
merged 4 commits into from
Jan 20, 2025

Conversation

maxrjones
Copy link
Member

I found the html coverage report simpler to use for monitoring progress in #2693 and am wondering if you would consider a hatch command for this, such as the one suggested in the PR.

I also propose adding a run-pytest alias for run because I heard that using hatch run --env <env> run to run the tests was a bit confusing to new contributors at the Pangeo hackday.

@d-v-b
Copy link
Contributor

d-v-b commented Jan 19, 2025

+1 for an easy path to making an html coverage report. no strong feelings about the new alias for run, but if people found it confusing, we should definitely fix that.

Copy link
Contributor

@dstansby dstansby left a comment

Choose a reason for hiding this comment

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

Thanks, this is a nice addition!

@dstansby dstansby merged commit e9772ac into zarr-developers:main Jan 20, 2025
30 checks passed
@maxrjones maxrjones deleted the minor-contrib-improvement branch January 20, 2025 14:45
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.

3 participants