Skip to content

Quote python-version value in the tests workflow - #6882

Merged
albertvillanova merged 1 commit into
mainfrom
quote-python-version-tests
Aug 24, 2026
Merged

Quote python-version value in the tests workflow#6882
albertvillanova merged 1 commit into
mainfrom
quote-python-version-tests

Conversation

@albertvillanova

@albertvillanova albertvillanova commented Aug 24, 2026

Copy link
Copy Markdown
Member

This PR quotes the python-version value in the tests workflow, making it consistent with every other setup-python usage in the repository.

Follow-up to #6881.

Motivation

In YAML, an unquoted 3.12 is parsed as a float rather than a string. It happens to work here, but it is the classic pitfall behind versions such as 3.10 silently becoming 3.1. Every other setup-python usage in the repository already quotes the value.

Changes

  • Quote the python-version value in the code quality job of the tests workflow, the only remaining unquoted occurrence

Note

Low Risk
One-line CI YAML quoting change with no runtime or security impact.

Overview
Quotes python-version as '3.12' in the check_code_quality job of tests.yml.

This matches other setup-python usages and avoids YAML parsing the version as a float.

Reviewed by Cursor Bugbot for commit 05fec6c. Bugbot is set up for automated code reviews on this repo. Configure here.

@bot-ci-comment

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@albertvillanova

Copy link
Copy Markdown
Member Author

Merging this PR as trivial change.

@albertvillanova
albertvillanova merged commit 4453b53 into main Aug 24, 2026
10 checks passed
@albertvillanova
albertvillanova deleted the quote-python-version-tests branch August 24, 2026 08:57
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.

1 participant