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

ruff 0.8.0 fixes #24488

Merged
merged 4 commits into from
Nov 25, 2024
Merged

ruff 0.8.0 fixes #24488

merged 4 commits into from
Nov 25, 2024

Conversation

joar
Copy link

@joar joar commented Nov 25, 2024

@karthiknadig karthiknadig added the debt Covers everything internal: CI, testing, refactoring of the codebase, etc. label Nov 25, 2024
@karthiknadig karthiknadig self-assigned this Nov 25, 2024
@karthiknadig karthiknadig enabled auto-merge (squash) November 25, 2024 16:49
@vs-code-engineering vs-code-engineering bot added this to the November 2024 milestone Nov 25, 2024
@karthiknadig
Copy link
Member

We typically keep python 3.8 support for 3 months after deprecation. Feb release is when we do official deprecation. Till then we would need to keep this from breaking.

@karthiknadig karthiknadig merged commit 98fea6e into microsoft:main Nov 25, 2024
42 of 48 checks passed
@karthiknadig
Copy link
Member

@joar I am reverting this change since 3.8 broke with this change. I will restore it, after we deprecate 3.8 support in the extension (which will be in Feb 2025).

karthiknadig added a commit that referenced this pull request Nov 25, 2024
karthiknadig added a commit that referenced this pull request Nov 25, 2024
karthiknadig added a commit that referenced this pull request Nov 25, 2024
karthiknadig pushed a commit that referenced this pull request 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`](https://docs.astral.sh/ruff/settings/#target-version)
> or
[`project.requires-python`](https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#python-requires)
>
([https://github.com/microsoft/vscode-python/pull/13896](https://github.com/astral-sh/ruff/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.
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