Skip to content

Avoid syntax error from E502 fixes in f-strings and t-strings#24410

Merged
charliermarsh merged 1 commit intomainfrom
charlie/continuation-fix
Apr 5, 2026
Merged

Avoid syntax error from E502 fixes in f-strings and t-strings#24410
charliermarsh merged 1 commit intomainfrom
charlie/continuation-fix

Conversation

@charliermarsh
Copy link
Copy Markdown
Member

Summary

It looks like this code was special-casing TokenKind::String, but missed the non-TokenKind::String string-like tokens (like for f-strings).

Closes #24409.

@charliermarsh charliermarsh added the bug Something isn't working label Apr 5, 2026
@charliermarsh charliermarsh force-pushed the charlie/continuation-fix branch from ff6750e to d50b2b6 Compare April 5, 2026 01:13
@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot bot commented Apr 5, 2026

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@charliermarsh charliermarsh marked this pull request as ready for review April 5, 2026 01:25
@charliermarsh charliermarsh merged commit 0abbde5 into main Apr 5, 2026
43 checks passed
@astral-sh-bot astral-sh-bot bot requested a review from amyreese April 5, 2026 01:25
@charliermarsh charliermarsh deleted the charlie/continuation-fix branch April 5, 2026 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix error] Fix for E502 introduces syntax error

2 participants