Respect diagnostic start and parent ranges and trailing comments in `ruff:ignore` suppressions #25673
Allow `ruff:ignore` to suppress diagnostics at the start of their ranges
e360affe
add an mdtest with todo
78bf5f2e
check contains(start) instead of contains_range
ad5d04a1
handle empty ranges
5c54964d
exclude range suppressions from the start check
faabb2c7
add a test for parenthesization issue
2b2b991e
also add standalone test
74276ef8
ntBre
marked this pull request as ready for review 136 days ago
add a couple of todo tests from micha's review
8aa879c0
extend own-line suppression ranges to include comments
5d75c382
add a few additional test cases
d02b350c
also check parent ranges
732069f6
move parent check into ignore branch, try to simplify check slightly
480ad338
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
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
drop contains_inclusive checks now that contains works
2a53ab04
split up condition and add inline comment
6df4db9a
add tests for parent/non-parent suppressions
38375843
handle trailing whitespace
b83ab43b
add a couple mdtest comments
855ce9a8
typo
48545cd0
ntBre
merged
b78e92bf
into main 132 days ago
ntBre
deleted the brent/eol-ruff-ignore branch 132 days ago
Assignees
No one assigned
Labels
suppression
preview
Login to write a write a comment.
Login via GitHub