KEMBAR78
ruff 0.8.0 fixes by joar · Pull Request #24488 · microsoft/vscode-python · GitHub
Skip to content

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