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

Support ruff 0.8.0 by setting ruff target-version to py38 #24492

Merged
merged 3 commits into from
Nov 26, 2024

Conversation

joar
Copy link

@joar joar commented Nov 26, 2024

ruff 0.8.0 (released 2024-11-22) no longer defaults to supporting Python 3.8,

Ruff now defaults to Python 3.9 instead of 3.8 if no explicit Python version
is configured using ruff.target-version
or project.requires-python
(https://github.com/microsoft/vscode-python/pull/13896)
https://github.com/astral-sh/ruff/blob/f3dac27e9aa6ac6a20fc2fb27ff2e4f5d369b076/CHANGELOG.md#080

We want to support Python 3.8 until February 2025, so we need to set
target-version.

The minimum Python version to target, e.g., when considering automatic code
upgrades, like rewriting type annotations. Ruff will not propose changes
using features that are not available in the given version.
https://docs.astral.sh/ruff/settings/#target-version

Can be used as an alternative to #24488 until py38 support is dropped.

This PR also reverts the pinning of ruff merged in #24484.

joar added 3 commits November 26, 2024 11:04
`ruff` 0.8.0 (released 2024-11-22) no longer defaults to supporting Python 3.8,

> Ruff now defaults to Python 3.9 instead of 3.8 if no explicit Python version
> is configured using [`ruff.target-version`](https://docs.astral.sh/ruff/settings/#target-version)
> or [`project.requires-python`](https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#python-requires)
> ([microsoft#13896](astral-sh/ruff#13896))
> — https://github.com/astral-sh/ruff/blob/f3dac27e9aa6ac6a20fc2fb27ff2e4f5d369b076/CHANGELOG.md#080

We want to support Python 3.8 until February 2025, so we need to set
`target-version`.

> The minimum Python version to target, e.g., when considering automatic code
> upgrades, like rewriting type annotations. Ruff will not propose changes
> using features that are not available in the given version.
> — https://docs.astral.sh/ruff/settings/#target-version
@vs-code-engineering vs-code-engineering bot added this to the November 2024 milestone Nov 26, 2024
@karthiknadig karthiknadig self-assigned this Nov 26, 2024
@karthiknadig karthiknadig added the debt Covers everything internal: CI, testing, refactoring of the codebase, etc. label Nov 26, 2024
@karthiknadig karthiknadig enabled auto-merge (squash) November 26, 2024 11:36
@karthiknadig karthiknadig merged commit 42b63b9 into microsoft:main Nov 26, 2024
47 of 48 checks passed
@joar joar deleted the life-is-ruff-when-you-support-py3.8 branch November 27, 2024 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debt Covers everything internal: CI, testing, refactoring of the codebase, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants