Repository navigation
chore: drop the MANIFEST.in lines that match nothing (#1794) - #1813
Conversation
- global-include *.rst: every tracked .rst is under docs/, which prune docs drops. Only with a conda _extern tree did it ship anything: two vendored files, aenum/doc/aenum.rst and html5lib's dist-info AUTHORS.rst, neither read at runtime. - include pcapkit/_extern: include matches files, not a directory. The conda recipe runs pip install . on the tree and builds no sdist; _extern's modules reach the wheel through package discovery. - exclude pcapkit/protocols/*/NotImplemented: the placeholders are directories; they ship either way, as documentation.rst records. - MANIFEST.in comments and documentation.rst stop citing *.rst. - ManifestTests: dropping any non-prune line must change what the template selects; on origin/main it fails on exactly these lines. sdist 1249 and wheel 534 entries, unchanged; both warnings gone. tests/project: 951 passed, 1 skipped. Closes #1794
|
Verdict: GOOD TO GO at |
|
Coverage: 89.60% (unit tier, Python 3.15,
Per-file detail: the |
|
|
|
Yes,
So conda builds today get the vendored |
Please follow the guide below
You will be asked some questions, please read them carefully and answer honestly
Put an
xinto all the boxes [ ] relevant to your pull request (like that [x])Use Preview tab to see how your pull request will actually look like
Searched for similar pull requests
Followed the coding style (
make pylint,make mypy,make isort)make testpasses, and a test case covers the change. Narrow runs only, listed below.Added a changelog entry under
docs/source/changelog/and regeneratedCHANGELOG.md, if the change is user-visible: N/A — centralised in docs(changelog): shared 1.5.0 changelog — long-lived, merges last (#610, #616, #617, #618, #620) #657What is the purpose of your pull request?
Tick the commit type your subject line carries.
fix— corrects a defectfeat— adds a featureperf— changes performance, not behaviourrefactor— changes neither behaviour nor performancetest— tests onlydocs— documentation onlyci— workflows or build toolingrelease— bumps the version or rolls up a distributionchore— anything elseSummary
MANIFEST.inhad lines that matched nothing, as #1794 measured. All are removed; none needed correcting.global-include *.rst: every tracked.rstis underdocs/, whichprune docsdrops.include pcapkit/_extern:includematches files, not a directory. The conda recipe builds the wheel straight from the tree withpip install ., never an sdist, and the vendored modules reach the wheel through package discovery either way.exclude pcapkit/protocols/*/NotImplemented: the placeholders are directories, so the pattern matched nothing. They ship either way, asdocumentation.rstsays._extern, only two vendored.rstfiles that nothing reads drop out.tests/project/test_build_leftovers_1773.pygains a guard that every non-prunedirective changes the selection; it fails onmainon exactly these lines.Closes #1794