build(deps): require packaging>=24.0 and drop pytest-accept by d-v-b · Pull Request #4626 · zarr-developers/zarr-python

d-v-b · 2026-11-12T13:20:42Z

two follow-up fixes after #4624. we bump packaging floor to 24.0 and drop pytest-accept which isn't used any more, and can always be used by a motivated developer by installing it directly.

🤖 AI text below 🤖

The minimum supported packaging version is now 24.0.

This draft builds on #4624 and replaces two of its workarounds for pytest 9.2 with simpler fixes: it drops the unused pytest-accept plugin and raises the packaging floor.

Changes

Drop pytest-accept. docs: split tutorial into multiple user guide sections #2589 added it for a hatch run doctest:fix script that ran pytest --accept over the RST doctests. That script is gone, and nothing in the justfile or CI passes --accept. The plugin's only remaining effect was importing astor (last released in 2019) at every pytest startup. Removing it removes astor, so chore(deps-dev): bump the python-dependencies group (13 updates) and keep the suite running with pytest 9.2 #4624's astor-scoped filterwarnings entry is deleted too. It is also removed from ci/big-endian/Dockerfile.
Require packaging>=24.0. pytest 9.2 requires packaging>=24, which is why chore(deps-dev): bump the python-dependencies group (13 updates) and keep the suite running with pytest 9.2 #4624 held min_deps on pytest<9.2 through UV_OVERRIDE and ci/min_deps-overrides.txt. This PR raises the runtime floor instead (packaging 24.0 was released in March 2024) and pins min_deps to packaging==24.*. zarr uses only packaging.version.Version and parse, which behave the same on 22 and 24. min_deps now runs the same pytest as every other environment, and the override file and env var are removed. The installation guide is updated to match.

Testing

Python 3.13, pytest 9.2.0, from uv.lock: astor and pytest_accept are not installed, and the full suite collects (14,529 tests) with no filterwarnings entry for astor.
min_deps recreated with hatch 1.16.5 (the version CI uses): pytest 9.2.0, packaging 24.2, UV_OVERRIDE unset. Full suite on macOS (pytest -n 8 --ignore tests/benchmarks): 13,278 passed, 1,268 skipped, 9 xfailed, 0 failed.
just lint (prek, including mypy) passes, and uv lock changes only the packaging specifier and the removed astor/pytest-accept entries.

Stack and merge order

chore(deps-dev): bump the python-dependencies group (13 updates) and keep the suite running with pytest 9.2 #4624's head branch is on a third-party fork, so this PR cannot be stacked on it natively. It targets main and contains chore(deps-dev): bump the python-dependencies group (13 updates) and keep the suite running with pytest 9.2 #4624's two commits until chore(deps-dev): bump the python-dependencies group (13 updates) and keep the suite running with pytest 9.2 #4624 merges.
Merge chore(deps-dev): bump the python-dependencies group (13 updates) and keep the suite running with pytest 9.2 #4624 first. The agent will then rebase this branch onto main, leaving only the two commits here.

🤖 Generated with Claude Code

pytest 9.2 applies the configured filterwarnings while importing plugins (pytest#12697), so the "error" filter turns the ast.Num DeprecationWarning that astor (a dependency of pytest-accept) triggers at import on Python < 3.14 into a startup failure (exit code 3). Ignore that one astor-scoped deprecation, which pytest <= 9.1 never routed through the filters. pytest 9.2 also requires packaging>=24, which conflicts with the min_deps pin packaging==22.* (the declared runtime floor packaging>=22.0). Keep that floor tested by overriding pytest to <9.2 in the min_deps env only, via UV_OVERRIDE and ci/min_deps-overrides.txt; every other env uses pytest 9.2.0. hypothesis 6.170.7 annotates run_state_machine_as_test as taking type[RuleBasedStateMachine] (its docstring still accepts any zero-arg factory), so mypy now reports arg-type instead of no-untyped-call at the four call sites that pass a factory function. Update the ignore codes; no test logic changes.

pytest-accept was added in zarr-developers#2589 for a `hatch run doctest:fix` script that ran `pytest --accept` over the RST doctests. That script and those doctests are gone, and nothing in the justfile or CI passes `--accept`. The plugin's only remaining effect was importing astor (last released in 2019) at every pytest startup, whose `ast.Num` lookup pytest 9.2 turns into a startup error under our "error" warning filter. Removing the plugin removes astor, so the astor-scoped filter added for pytest 9.2 is no longer needed. Assisted-by: ClaudeCode:claude-opus-5-5

pytest 9.2 requires packaging>=24, so the min_deps env could no longer pin packaging==22.*. Instead of holding min_deps on pytest<9.2 through a UV_OVERRIDE file, raise the runtime floor to packaging>=24.0 (released March 2024) and test that floor in min_deps. zarr uses only packaging.version.Version and parse, which behave the same on 22 and 24. min_deps now runs the same pytest as every other env, and the override file and env var are gone. Assisted-by: ClaudeCode:claude-opus-5-5

Assisted-by: ClaudeCode:claude-opus-5-5

codecov · 2026-11-12T14:01:06Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.90%. Comparing base (901290d) to head (f57591f).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4626   +/-   ##
=======================================
  Coverage   94.90%   94.90%           
=======================================
  Files          96       96           
  Lines       14048    14048           
=======================================
  Hits        13332    13332           
  Misses        716      716

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…ccept-packaging-24 # Conflicts: # uv.lock

dependabot Bot and others added 6 commits November 12, 2026 04:31

docs: add a changelog entry for the packaging>=24.0 floor

55bacb9

Assisted-by: ClaudeCode:claude-opus-5-5

Merge branch 'main' into chore/drop-pytest-accept-packaging-24

df20372

d-v-b mentioned this pull request Nov 12, 2026

fix(examples): run the example scripts without pytest #4625

Merged

Merge remote-tracking branch 'upstream/main' into chore/drop-pytest-a…

e82486a

…ccept-packaging-24 # Conflicts: # uv.lock

d-v-b marked this pull request as ready for review November 12, 2026 18:08

Merge branch 'main' into chore/drop-pytest-accept-packaging-24

f57591f

d-v-b merged commit 84a4992 into zarr-developers:main Nov 12, 2026
38 checks passed

Navigation Menu

Repository navigation

build(deps): require packaging>=24.0 and drop pytest-accept - #4626

build(deps): require packaging>=24.0 and drop pytest-accept#4626
d-v-b merged 8 commits into
zarr-developers:mainfrom
d-v-b:chore/drop-pytest-accept-packaging-24

d-v-b commented Nov 12, 2026 •

edited

Loading

Uh oh!

codecov Bot commented Nov 12, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

d-v-b commented Nov 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codecov Bot commented Nov 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

d-v-b commented Nov 12, 2026 •

edited

Loading

codecov Bot commented Nov 12, 2026 •

edited

Loading