ruff
Respect diagnostic start and parent ranges and trailing comments in `ruff:ignore` suppressions
#25673
Merged

Respect diagnostic start and parent ranges and trailing comments in `ruff:ignore` suppressions #25673

ntBre merged 18 commits into main from brent/eol-ruff-ignore
ntBre
ntBre Allow `ruff:ignore` to suppress diagnostics at the start of their ranges
e360affe
ntBre add an mdtest with todo
78bf5f2e
ntBre check contains(start) instead of contains_range
ad5d04a1
ntBre handle empty ranges
5c54964d
ntBre exclude range suppressions from the start check
faabb2c7
ntBre add a test for parenthesization issue
2b2b991e
ntBre also add standalone test
74276ef8
ntBre ntBre added suppression
ntBre ntBre added preview
astral-sh-bot
ntBre ntBre marked this pull request as ready for review 136 days ago
MichaReiser MichaReiser requested a review from MichaReiser MichaReiser 133 days ago
MichaReiser
MichaReiser commented on 2026-06-08
ntBre add a couple of todo tests from micha's review
8aa879c0
ntBre extend own-line suppression ranges to include comments
5d75c382
ntBre add a few additional test cases
d02b350c
ntBre also check parent ranges
732069f6
ntBre move parent check into ignore branch, try to simplify check slightly
480ad338
ntBre
ntBre ntBre requested a review from MichaReiser MichaReiser 133 days ago
MichaReiser
ntBre ntBre changed the title Allow `ruff:ignore` to suppress diagnostics at the start of their ranges Fix divergence between `noqa` and `ruff:ignore` 133 days ago
ntBre ntBre changed the title Fix divergence between `noqa` and `ruff:ignore` Respect diagnostic start and parent ranges and trailing comments in `ruff:ignore` suppressions 133 days ago
ntBre drop contains_inclusive checks now that contains works
2a53ab04
ntBre
codspeed
MichaReiser
MichaReiser approved these changes on 2026-06-09
ntBre split up condition and add inline comment
6df4db9a
ntBre add tests for parent/non-parent suppressions
38375843
ntBre handle trailing whitespace
b83ab43b
ntBre add a couple mdtest comments
855ce9a8
ntBre typo
48545cd0
ntBre ntBre merged b78e92bf into main 132 days ago
ntBre ntBre deleted the brent/eol-ruff-ignore branch 132 days ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
Milestone