-
Notifications
You must be signed in to change notification settings - Fork 56
[GHA] Add new setup-python action #983
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
base: main
Are you sure you want to change the base?
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 would prefer avoiding uv
. This is not used in the Swift.org builds, and diverging the builds further makes it harder to maintain. We already try to synchronise the build between build.ps1 and GHA, adding a diff on top is more complicated. Why can we not just fetch python if we want to use our own copy?
UV is doing that for us. we are still using |
Yes, I realize what |
2d08d0d
to
2d02b11
Compare
@compnerd reverted back to github's |
Please hold off on merging this. i need to get a successful downstream run first. |
0067bf3
to
dba409e
Compare
Add a new GH Action to setup python using
uv
. Also adding explicit calls to setup uv where we are executing python scripts.Downstream CI run with an image not including python (thebrowsercompany/swift-build#302):
https://github.com/thebrowsercompany/swift-build/actions/runs/15982191821