Combine build-tool update and dependency install into one CI step per job#12643
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #12643 +/- ##
=======================================
Coverage 98.95% 98.95%
=======================================
Files 131 131
Lines 46476 46476
Branches 2410 2410
=======================================
Hits 45988 45988
Misses 366 366
Partials 122 122
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Merging this PR will not alter performance
Comparing Footnotes
|
Backport to 3.13: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 348f0a1 on top of patchback/backports/3.13/348f0a1ed2887662822a44ca0f6ec0668aa292da/pr-12643 Backporting merged PR #12643 into master
🤖 @patchback |
Backport to 3.14: 💚 backport PR created✅ Backport PR branch: Backported as #12645 🤖 @patchback |
…endency install into one CI step per job (#12645)
What do these changes do?
Merges the "Update pip, wheel, setuptools, build, twine" step into the following "Install dependencies" (or "Install cython") step in every job in
ci-cd.yml; one resolver invocation per job instead of two, with the same set of packages getting installed.Are there changes in behavior for the user?
No; same packages are still installed, just in a single
pip/uv pip installcall. The "Install self" steps are intentionally left alone since they run aftermake cythonizeand rely on built extensions.Is it a substantial burden for the maintainers to support this?
No; it is a mechanical collapse across seven jobs, easy to revert per-job if anything regresses.
Related issue number
Follow-up to #12641, continuing the work to make the CI install path cheaper for downstream consumers using uv.
Checklist
CONTRIBUTORS.txtCHANGES/folderCHANGES/12643.contrib.rst.Drafted with Claude Code (claude-opus-4-7); reviewed by @bdraco.