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

fix: download pipx for action, allow support for M1 #1743

Merged
merged 5 commits into from Jan 30, 2024

Conversation

henryiii
Copy link
Contributor

Takes an extra second or two, plus it could interfere with the environment a bit if someone activates the same version of Python. But not too bad.

@joerick
Copy link
Contributor

joerick commented Jan 30, 2024

@henryiii, hope it's okay, I pushed a couple commits to get this working. I hope that actions/setup-python wouldn't put spaces in the path! I want to do a quick release to get that Powershell fix in and I thought that we might as well let people try out the new macos-14 runners. We can do the official support with examples/docs in #1700.

@joerick
Copy link
Contributor

joerick commented Jan 30, 2024

Erm, actually, on second thoughts maybe that's not a good assumption given 'Program Files' and all that.

@henryiii
Copy link
Contributor Author

Great, thanks! Had to teach. That's what I was thinking, fixing the powershell plus macOS AS runners would be a good quick patch release.

@henryiii
Copy link
Contributor Author

This won't break Travis, so fine to merge IMO.

@henryiii henryiii merged commit 5b0b458 into pypa:main Jan 30, 2024
17 of 18 checks passed
@henryiii henryiii deleted the henryiii/fix/action3 branch January 30, 2024 23:00
@joerick
Copy link
Contributor

joerick commented Jan 30, 2024

brill, thanks, releasing now...

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