Skip to content

Remove files from MANIFEST.in#12041

Merged
jacobtomlinson merged 2 commits intodask:mainfrom
DimitriPapadopoulos:MANIFEST.in
Nov 4, 2025
Merged

Remove files from MANIFEST.in#12041
jacobtomlinson merged 2 commits intodask:mainfrom
DimitriPapadopoulos:MANIFEST.in

Conversation

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor

@DimitriPapadopoulos DimitriPapadopoulos commented Aug 2, 2025

setup.py, README.rst or MANIFEST.in are included in the sdist by default, like pyproject.toml or license files.

It looks versioneer.py has been removed.

  • Closes #xxxx
  • Tests added / passed
  • Passes pre-commit run --all-files

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 2, 2025

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

      9 files  ±0        9 suites  ±0   3h 7m 9s ⏱️ +43s
 18 139 tests ±0   16 924 ✅ ±0   1 215 💤 ±0  0 ❌ ±0 
162 479 runs  ±0  150 386 ✅ ±0  12 093 💤 ±0  0 ❌ ±0 

Results for commit dc279c1. ± Comparison against base commit 17a245a.

♻️ This comment has been updated with latest results.

@DimitriPapadopoulos DimitriPapadopoulos changed the title Remove MANIFEST.in from MANIFEST.in Remove files from MANIFEST.in Aug 2, 2025
@github-actions github-actions bot added the needs attention It's been a while since this was pushed on. Needs attention from the owner or a maintainer. label Sep 15, 2025
@github-actions github-actions bot removed the needs attention It's been a while since this was pushed on. Needs attention from the owner or a maintainer. label Oct 27, 2025
The file is missing from the Git repository, it cannot be added to the sdist.

I have checked that versioneer.py is missing from the latest sdist on PyPI.
Copy link
Copy Markdown
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

Did you make these changes based on some automation? Or just through manual inspection?

@jacobtomlinson jacobtomlinson merged commit 88fd00d into dask:main Nov 4, 2025
23 of 24 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the MANIFEST.in branch November 4, 2025 11:11
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