Skip to content

Clean up test extra dependencies#3656

Merged
d-v-b merged 3 commits intozarr-developers:mainfrom
dstansby:test-extra
Jan 16, 2026
Merged

Clean up test extra dependencies#3656
d-v-b merged 3 commits intozarr-developers:mainfrom
dstansby:test-extra

Conversation

@dstansby
Copy link
Contributor

I noticed that there were a few dependencies under [test] that were superfluous, since this extra should contain only dependencies for running the tests, not optional extras. This allows the min_deps run on GH actions to then truly be a minimal dependencies run.

Because it looked like some of these dependencies were useful developement dependencies, I added a new [dev] extra that is helpful for developers who just want to install everything useful for development.

@github-actions github-actions bot added needs release notes Automatically applied to PRs which haven't added release notes and removed needs release notes Automatically applied to PRs which haven't added release notes labels Jan 15, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 15, 2026

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing dstansby:test-extra (fc4a863) with main (4cddcf5)

Summary

✅ 30 untouched benchmarks

@dstansby dstansby marked this pull request as ready for review January 15, 2026 20:06
@d-v-b
Copy link
Contributor

d-v-b commented Jan 15, 2026

We need uv for regression tests

Co-authored-by: Davis Bennett <davis.v.bennett@gmail.com>
@d-v-b d-v-b merged commit e94504e into zarr-developers:main Jan 16, 2026
25 checks passed
@dstansby dstansby deleted the test-extra branch January 17, 2026 20:15
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