Skip to content
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

Remove sudo from uv configuration #157

Merged
merged 1 commit into from Apr 21, 2024
Merged

Conversation

charliermarsh
Copy link
Contributor

Summary

It looks like the issue here is that the workflow was missing actions/setup-python. Without it, I think the workflow has access to Python in the "tool cache" but that it's like... not fully configured? So you don't have write permissions to it?

pip has a behavior whereby if you try to install, and you don't have permissions to write, it falls back to a slightly different setup (as if you ran pip install --user). We don't have that behavior in uv. But if you use setup-python (is 0 seconds anyway), then the workflow will have the expected permissions, and uv pip install --system works without sudo.

Test Plan

@matsonj matsonj merged commit bac774f into matsonj:master Apr 21, 2024
1 check passed
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.

None yet

2 participants