Skip to content

Avoid introducing syntax errors for FAST003 autofix#23227

Merged
charliermarsh merged 1 commit intomainfrom
charlie/fa-fix
Feb 11, 2026
Merged

Avoid introducing syntax errors for FAST003 autofix#23227
charliermarsh merged 1 commit intomainfrom
charlie/fa-fix

Conversation

@charliermarsh
Copy link
Member

Closes #19831.

@charliermarsh charliermarsh added bug Something isn't working fixes Related to suggested fixes for violations labels Feb 11, 2026
@charliermarsh charliermarsh marked this pull request as ready for review February 11, 2026 13:41
@astral-sh-bot
Copy link

astral-sh-bot bot commented Feb 11, 2026

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

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.

I think we're still missing coverage for the __debug__ and NFKC cases from the issue, so we may want to leave it open, but this looks good to me otherwise!

@charliermarsh charliermarsh enabled auto-merge (squash) February 11, 2026 14:48
@charliermarsh charliermarsh merged commit 38fa994 into main Feb 11, 2026
41 checks passed
@charliermarsh charliermarsh deleted the charlie/fa-fix branch February 11, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working fixes Related to suggested fixes for violations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FAST003 fix introduces syntax errors

2 participants