Skip to content

Conversation

@jacobtomlinson
Copy link
Member

@jacobtomlinson jacobtomlinson commented Nov 4, 2025

Given that setuptools-scm is more widely used and better maintained than versioneer (last release was July 2023) we might want to switch.

  • Updated pyproject.toml to use setuptools-scm
  • Removed the distributed/_version.py file as setuptools-scm generates this during sdist/wheel build (or pip install -e .)
  • Removed unused setup.py
  • Removed __getattr__ shim added in lazily get dask version information #5822.
    • The setuptools-scm implementation of _version.py should be much faster than the way versioneer was generating from git every import so it's better to simplify things

xref dask/dask#12133

"get_client",
"get_task_metadata",
"get_task_stream",
"get_versions",
Copy link
Member Author

@jacobtomlinson jacobtomlinson Nov 4, 2025

Choose a reason for hiding this comment

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

Technically this is breaking, but this internal versioneer method was never intended to be exposed publicly anyway. If anyone is depending on this they should probably switch to __version__ or importlib.metadata.version("distributed").

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

Unit Test Results

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

    27 files  ±0      27 suites  ±0   9h 42m 32s ⏱️ +56s
 4 112 tests ±0   4 007 ✅ +1    104 💤 ±0  1 ❌  - 1 
51 516 runs  ±0  49 331 ✅ +2  2 184 💤  - 1  1 ❌  - 1 

For more details on these failures, see this check.

Results for commit b578cf6. ± Comparison against base commit a9ad6d4.

♻️ This comment has been updated with latest results.

@jacobtomlinson jacobtomlinson merged commit ca5a56f into dask:main Nov 6, 2025
30 of 34 checks passed
DimitriPapadopoulos added a commit to DimitriPapadopoulos/distributed that referenced this pull request Nov 15, 2025
File setup.py was removed by dask#9137 / ca5a56f.
DimitriPapadopoulos added a commit to DimitriPapadopoulos/distributed that referenced this pull request Nov 17, 2025
File setup.py was removed by dask#9137 / ca5a56f.
DimitriPapadopoulos added a commit to DimitriPapadopoulos/distributed that referenced this pull request Nov 19, 2025
File setup.py was removed by dask#9137 / ca5a56f.
DimitriPapadopoulos added a commit to DimitriPapadopoulos/distributed that referenced this pull request Nov 25, 2025
File setup.py was removed by dask#9137 / ca5a56f.
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.

1 participant