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

Use python 3.11 for Arch install tests #17947

Merged
merged 1 commit into from
May 2, 2024
Merged

Use python 3.11 for Arch install tests #17947

merged 1 commit into from
May 2, 2024

Conversation

emlowe
Copy link
Contributor

@emlowe emlowe commented May 1, 2024

On arch, we have to install a version of python since it doesn't come with it by default. We were using 3.9 which remains a reasonable choice - except that a new but in setuptools_scm results in CI failures (see pypa/setuptools-scm#1038)

Using 3.11 should solve this and also seems a reasonable choice and this should resolve the CI failure in the arch install test.

This failure is related to git 2.45.0 and setuptools_scm and not specifically arch - however arch is often the canary on updating to the latest packages. It would be seen on other linuxes as well with that version of git

@emlowe emlowe added Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes CI CI changes Changed Required label for PR that categorizes merge commit message as "Changed" for changelog labels May 1, 2024
@emlowe emlowe changed the title Update test-install-scripts.yml Use python 3.11 for Arch install tests May 1, 2024
@emlowe emlowe marked this pull request as ready for review May 1, 2024 22:57
@emlowe emlowe requested a review from a team as a code owner May 1, 2024 22:57
@pmaslana pmaslana merged commit efde01f into main May 2, 2024
352 of 353 checks passed
@pmaslana pmaslana deleted the EL-arch-maybe branch May 2, 2024 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog CI CI changes Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants