Skip to content

Conversation

@soundsonacid
Copy link
Contributor

@soundsonacid soundsonacid commented Jul 14, 2025

fixes #19326

@soundsonacid soundsonacid marked this pull request as ready for review July 14, 2025 19:59
@github-actions
Copy link
Contributor

github-actions bot commented Jul 14, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@ntBre ntBre self-requested a review July 14, 2025 21:15
@ntBre ntBre added bug Something isn't working fixes Related to suggested fixes for violations labels Jul 14, 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 for working on this! I had a small suggestion and also found a helpful-looking Indexer method that might simplify things a bit.

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.

PLR2044 fix ignores a backslash on the preceding line

2 participants