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.

Sign up for GitHub

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

Jump to bottom

#14393: Use python3 #15867

Merged
merged 1 commit into from
Dec 10, 2024
Merged

#14393: Use python3 #15867

merged 1 commit into from
Dec 10, 2024

Conversation

afuller-TT
Copy link
Contributor

@afuller-TT afuller-TT commented Dec 10, 2024 •
edited
Loading

Ticket

Progress towards #14393

Problem description

Not all systems can be assumed to have python. Call out python3 as that's what we need.

What's changed

env python -> env python3

Checklist

@afuller-TT
Use python3
ec88a3d 
Works better on systems that don't have a 'python' symlink
@afuller-TT afuller-TT requested review from yieldthought, uaydonat and a team as code owners December 10, 2024 16:19
@afuller-TT afuller-TT mentioned this pull request Dec 10, 2024
Merged
5 tasks
@afuller-TT afuller-TT merged commit d679e66 into main Dec 10, 2024
120 checks passed
@afuller-TT afuller-TT deleted the afuller/python3 branch December 10, 2024 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yieldthought yieldthought Awaiting requested review from yieldthought yieldthought is a code owner

@uaydonat uaydonat Awaiting requested review from uaydonat uaydonat is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@afuller-TT