Skip to content

Comments

[ruff 0.8] Remove deprecated rule UP027#14382

Merged
AlexWaygood merged 1 commit intoruff-0.8from
remove-up027
Nov 16, 2024
Merged

[ruff 0.8] Remove deprecated rule UP027#14382
AlexWaygood merged 1 commit intoruff-0.8from
remove-up027

Conversation

@AlexWaygood
Copy link
Member

@AlexWaygood AlexWaygood commented Nov 16, 2024

Summary

Remove UP027 in Ruff 0.8. UP027 has been deprecated since #12843, which landed as part of Ruff 0.6 (released on August 15th).

Test Plan

  • cargo test -p ruff_linter passes
  • git grep UP027, git grep UnpackedListComprehension and git grep unpacked_list_comprehension all show no references to the rule remain outside of the changelog
  • Wait for the ecosystem report to see how much impact this has

@AlexWaygood AlexWaygood added rule Implementing or modifying a lint rule breaking Breaking API change labels Nov 16, 2024
@AlexWaygood AlexWaygood marked this pull request as ready for review November 16, 2024 18:00
@AlexWaygood
Copy link
Member Author

Wait for the ecosystem report to see how much impact this has

Seems like it's not possible to get an ecosystem report here, because of the fact that this PR targets the ruff-0.8 branch, and will be the first commit on the ruff-0.8 branch that deviates from main. Once this is merged into the ruff-0.8 branch, I'll open a PR to merge that branch into main and take a look at the ecosystem report on that PR.

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

Labels

breaking Breaking API change rule Implementing or modifying a lint rule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants