Skip to content

fix(release): install hypothesis in cibuildwheel test env - #57

Merged
stffns merged 1 commit into
mainfrom
fix/release-hypothesis-dep
Apr 20, 2026
Merged

stffns merged 1 commit into
mainfrom
fix/release-hypothesis-dep

Conversation

@stffns

@stffns stffns commented Apr 20, 2026

Copy link
Copy Markdown
Owner

Summary

The v0.10.0 tag build failed on every wheel-test step:

```
E ModuleNotFoundError: No module named 'hypothesis'
```

Cause: PR #50 added `tests/test_properties.py`, which imports
`hypothesis`, but `release.yml` still only installs `pytest` in
the cibuildwheel test env.

Four wheel jobs (ubuntu, ubuntu-arm, macos-14, windows) all fail at
the inner `pytest {project}/tests -q` step. The `publish` job is
gated on `needs: [build_wheels, build_sdist]`, so nothing was
published to PyPI.

Fix

```yaml
CIBW_TEST_REQUIRES: "pytest hypothesis>=6.100"
```

Mirrors `pyproject.toml`'s `[dev]` pin.

Recovery path

Once this PR merges:

  1. Delete the v0.10.0 tag (both local and remote -- no wheels were
    published, so the retag is clean):
    ```bash
    git tag -d v0.10.0
    git push origin :refs/tags/v0.10.0
    ```
  2. Re-tag v0.10.0 on the new main so the fixed workflow picks it up:
    ```bash
    git tag -a v0.10.0 -m "snapvec 0.10.0"
    git push origin v0.10.0
    ```

Both steps require Jay's explicit OK since they touch shared tag state.

Test plan

  • `release.yml` still parses (yaml valid)
  • CI green on this PR
  • After merge + retag, wheels build successfully and publish to PyPI

The v0.10.0 tag build failed on every wheel-test step with
ModuleNotFoundError: hypothesis.  Cause: PR #50 added
tests/test_properties.py, which imports hypothesis, but
CIBW_TEST_REQUIRES still only installed pytest.  Add hypothesis so
the wheel smoke test can collect every test file.

No wheels were published from v0.10.0; this fix goes in before the
retag so the next release attempt publishes cleanly.
Copilot AI review requested due to automatic review settings April 20, 2026 14:28
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Fixes the release workflow’s wheel test environment so cibuildwheel installs hypothesis, matching the project’s dev/test requirements and preventing tag builds from failing during pytest.

Changes:

  • Update CIBW_TEST_REQUIRES in the release workflow to include hypothesis>=6.100.
  • Align wheel-test dependencies with pyproject.toml’s [project.optional-dependencies].dev.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@stffns
stffns merged commit c939105 into main Apr 20, 2026
14 checks passed
@stffns
stffns deleted the fix/release-hypothesis-dep branch April 20, 2026 14:33
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