Skip to content

feat: install python with uv#1219

Draft
SMoraisAnsys wants to merge 15 commits into
mainfrom
feat/install-python-with-uv
Draft

feat: install python with uv#1219
SMoraisAnsys wants to merge 15 commits into
mainfrom
feat/install-python-with-uv

Conversation

@SMoraisAnsys
Copy link
Copy Markdown
Contributor

@SMoraisAnsys SMoraisAnsys commented Mar 20, 2026

As title says. Could be very usefull when one wants to use our actions in containers where python provided by actions/setup-python does not work directly.

The draft still needs to be updated to point to the main branch.

Associated to #1218

@github-actions github-actions Bot added the enhancement General improvements to existing features label Mar 20, 2026
@SMoraisAnsys SMoraisAnsys self-assigned this Mar 20, 2026
@SMoraisAnsys SMoraisAnsys marked this pull request as ready for review March 25, 2026 09:28
@SMoraisAnsys SMoraisAnsys requested a review from a team as a code owner March 25, 2026 09:28
@SMoraisAnsys
Copy link
Copy Markdown
Contributor Author

I'm switching this into PR to get feedback on it :D I was hoping to get some from #1218 but that wasn't the case :(

Copy link
Copy Markdown
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've go my doubts regarding this... why not reuse the "use-uv" input? If we use uv it should be a consistent usage IMO..

Adding optional inputs/flags is just going to introduce confusion

@SMoraisAnsys
Copy link
Copy Markdown
Contributor Author

I've go my doubts regarding this... why not reuse the "use-uv" input? If we use uv it should be a consistent usage IMO..

Adding optional inputs/flags is just going to introduce confusion

I'm fine with it but that will change the previous behavior of our action. The main usage of uv was through the idea of providing uv to install packages. If the majority agreeds to using uv's python when using uv (which makes total sense) I'll do the changes ;)

@ansys/pyansys-core any strong opinion on one approach or the other ?

@moe-ad
Copy link
Copy Markdown
Contributor

moe-ad commented Apr 13, 2026

If we can rename use-uv to something that unambiguously reflects that it only installs packages (maybe use-uv-for-package-installation ?), I personally wouldn't mind adding another input for python installation.

If not, we probably shouldn't expose another input . We can instead document this as an additional behavior for the use-uv option in the migration guide to give the change more visibility. What do you think @SMoraisAnsys?

@SMoraisAnsys
Copy link
Copy Markdown
Contributor Author

If we can rename use-uv to something that unambiguously reflects that it only installs packages (maybe use-uv-for-package-installation ?), I personally wouldn't mind adding another input for python installation.

If not, we probably shouldn't expose another input . We can instead document this as an additional behavior for the use-uv option in the migration guide to give the change more visibility. What do you think @SMoraisAnsys?

Fully moving to uv (python provider & package installation) might have side effects on certain workflows. Typically, I'm thinking about this issue astral-sh/python-build-standalone#596 but others might exist. We could probably bring that topic to one of our meeting. Leaving this PR as draft in the meantime.

@SMoraisAnsys SMoraisAnsys marked this pull request as draft April 14, 2026 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement General improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants