-
Notifications
You must be signed in to change notification settings - Fork 660
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use uv instead of pip in Supervisor #5152
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need the same: https://github.com/home-assistant/core/blob/dev/Dockerfile#L32
Warning Review failedThe pull request is closed. WalkthroughWalkthroughThe recent updates streamline the Docker setup by introducing the Changes
Sequence Diagram(s)No sequence diagrams are needed for these changes as they primarily involve environment setup and configuration adjustments. Tip AI model upgrade
|
59a01ab
to
0423c44
Compare
0423c44
to
c8b1baf
Compare
Proposed change
Switch supervisor over to
uv
installer instead ofpip
for performance and consistency with core.Depends on home-assistant/devcontainer#87 for startup script split in devcontainer. However this PR does not update devcontainer to use
uv
as that has issues which need to be resolved first.Type of change
Additional information
Checklist
ruff format supervisor tests
)If API endpoints of add-on configuration are added/changed:
Summary by CodeRabbit
UV_SYSTEM_PYTHON=true
environment variable and usinguv
for package management..devcontainer/devcontainer.json
to refine commands and settings for a more efficient development environment, including paths, testing, and linting configurations.