Skip to content

Conversation

dstansby
Copy link
Contributor

@dstansby dstansby commented Aug 4, 2025

Fixes #3333 (hopefully). To prevent me having to copy/paste environment variables for four different hatch environments, I also de-duplicated the configuration of the test environments. 🤞 this works fine, happy to revert the de-duplication and do a copy-and-paste job if folks would prefer that to keep things a bit simpler.

@github-actions github-actions bot added the needs release notes Automatically applied to PRs which haven't added release notes label Aug 4, 2025
@dstansby dstansby added this to the 3.1.2 milestone Aug 4, 2025
Copy link

codecov bot commented Aug 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.40%. Comparing base (1858af1) to head (a609d06).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3334       +/-   ##
===========================================
+ Coverage   60.69%   94.40%   +33.70%     
===========================================
  Files          78       78               
  Lines        9401     9401               
===========================================
+ Hits         5706     8875     +3169     
+ Misses       3695      526     -3169     

see 66 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dstansby dstansby merged commit 22b177c into zarr-developers:main Aug 4, 2025
31 checks passed
meeseeksmachine pushed a commit to meeseeksmachine/zarr-python that referenced this pull request Aug 4, 2025
@dstansby dstansby deleted the fix-coverage branch August 4, 2025 16:14
dstansby added a commit that referenced this pull request Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs release notes Automatically applied to PRs which haven't added release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mysterious code coverage drop
2 participants