Repository navigation
Commit 5abdee2
authored
build(zarr-metadata): the sdist ships an allowlist, not whatever is lying around (#4248)
hatchling had no sdist configuration here either, so a source distribution
carried everything in the package directory. Building from a working tree with
scratch files in it put `.env.local`, a notebook and a `__scratch/` dump in the
tarball. A tagged release builds from a fresh CI checkout and so was never
actually at risk, but nothing made that a property of the package rather than
of the runner.
The list is derived from this package rather than copied from zarr-indexing,
which needed `docs/snippets` and `examples/` because its suite executes them.
Nothing here does: every fixture is a JSON file next to the test module that
reads it, so `/tests` is the whole test dependency. `/docs` and `/mkdocs.yml`
ride along because they are self-contained — mkdocstrings reads `src` and the
config reaches nowhere outside the package — so the sdist documents itself as
well as tests itself. `changes/` and `.readthedocs.yaml` are left out: towncrier
fragments are repo bookkeeping, and the RTD config addresses paths from the
repo root, where an unpacked sdist is not.
Verified by unpacking the built sdist into a bare venv and working from there:
595 tests pass and `mkdocs build --strict` succeeds.
Assisted-by: ClaudeCode:claude-opus-51 parent f63e61d commit 5abdee2
1 file changed
Lines changed: 20 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
74 | 94 | | |
75 | 95 | | |
76 | 96 | | |
| |||
0 commit comments