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

Add support for .tool-versions file in setup-python #1043

Draft

mahabaleshwars wants to merge 1 commit into actions:main from mahabaleshwars:feature/tool-versions-file-support

+168 −6

mahabaleshwars commented Feb 24, 2025 •

edited

Loading

Description:
Added support for .tool-versions file in setup-python. It can be passed through python-version-file.

Related issue:
#970

Check list:

Mark if documentation changes are required.
Mark if tests were added or updated to cover the changes.


          add support for .tool-versions file

b79caba

github-advanced-security bot found potential problems

View reviewed changes

src/utils.ts

+                      continue;
+                    }
+                    const match = line.match(
+                      /^\s*python\s*v?(?<version>[^\s]+(?:\s*[-<>=!]+[^\s]+)*)\s*(-\s([^\s].*))?\s*$/

Check failure

Code scanning / CodeQL

Inefficient regular expression High

This part of the regular expression may cause exponential backtracking on strings starting with 'python!' and containing many repetitions of '!!'.

src/utils.ts

+                      continue;
+                    }
+                    const match = line.match(
+                      /^\s*python\s*v?(?<version>[^\s]+(?:\s*[-<>=!]+[^\s]+)*)\s*(-\s([^\s].*))?\s*$/

Check failure

Code scanning / CodeQL

Inefficient regular expression High

This part of the regular expression may cause exponential backtracking on strings starting with 'python!!' and containing many repetitions of '!!'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet