Skip to content

Comments

[ruff] Do not flag %r + repr() combinations (RUF065)#20600

Merged
ntBre merged 1 commit intoastral-sh:mainfrom
danparizher:fix-20583
Sep 30, 2025
Merged

[ruff] Do not flag %r + repr() combinations (RUF065)#20600
ntBre merged 1 commit intoastral-sh:mainfrom
danparizher:fix-20583

Conversation

@danparizher
Copy link
Contributor

@danparizher danparizher commented Sep 27, 2025

Summary

Fixes the first part of #20583

@github-actions
Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@ntBre ntBre added rule Implementing or modifying a lint rule preview Related to preview mode features labels Sep 30, 2025
Copy link
Contributor

@ntBre ntBre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I think it made sense to separate this from adding support for the other combinations in #20583.

I'll just unlink that issue from this to avoid closing it, and we can follow up with a separate PR expanding the rule.

@ntBre ntBre merged commit 7c87b31 into astral-sh:main Sep 30, 2025
38 checks passed
@danparizher danparizher deleted the fix-20583 branch September 30, 2025 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview Related to preview mode features rule Implementing or modifying a lint rule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants