Repository navigation
Use setup-uv defaults for Python and the venv - #127
Conversation
|
👋 Hello @glenn-jocher, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
Reviewed the publish workflow’s switch to setup-uv defaults and installation into the activated venv. The changed steps remain consistent with the workflow’s Python usage. LGTM.
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 2
Made with ❤️ by Ultralytics Actions
Since review 1, no additional code changes are shown. The Python setup and venv installation changes remain sound, with no findings. LGTM.
|
🎉 This PR has been merged—thank you, @glenn-jocher!
The publish workflow now installs |
Replaces
actions/setup-pythonwithultralytics/actions/setup-uv. By default, that action now activates a venv on the newest Python preinstalled on the runner, with no Python download. Packages install into that venv, so--systemis dropped. Jobs that pin a Python version keep the pin onsetup-uv, which uses the preinstalled copy when the runner has one.CI first ran against the
setup-uv-defaultsbranch of ultralytics/actions#915. #915 is now merged (da5463a, released as 0.4.0), sosetup-uvis back on@main, which carries the new defaults.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
The publish workflow now uses
ultralytics/actions/setup-uvinstead ofactions/setup-python, and installsultralytics-actionswithout the--systemflag.📊 Key Changes
actions/setup-pythonstep that selected Python3.x.ultralytics/actions/setup-uv@mainto set up Python and the virtual environment.--systemfrom theuv pip installcommand.🎯 Purpose & Impact
ultralytics-actionsinstalls into the environment activated bysetup-uvrather than the system Python environment.