Tatsh / deltona / 34200408651

100%

push

github

project: name Ruff rules instead of numbering them

Three Ruff rules were themselves being ignored, and they are the ones
that ask for this. noqa-comments rejects `# noqa` in favour of
`# ruff: ignore[...]`, rule-codes-in-suppression-comments wants the
rule's name inside it, and rule-codes-in-selectors wants names in the
configuration too. Dropping those three from the ignore list is what
produced every other change here.

A code has to be looked up. `# ruff: ignore[blind-except]` reads as the
thing the rule enforces, and so does an ignore list written out in
pyproject.toml. Ruff resolves either form, so the codes were only ever
the shorter of the two. output-prefer-rule-codes goes as well, so a
diagnostic reports the name a suppression would have to carry.

The `# noqa` form has a second problem: it cannot name a rule that has
no code, and pytest-fixture-autouse, which the test configuration
ignores, does not have one.

The lazy import in _decrypt_aes_cbc suppresses unsorted-imports too.
Spelling the rule out pushes the statement past the column limit, where
isort wants one name to a line and yapf joins them back, and neither
yields. The import is sorted; only the wrapping is in dispute.

Signed-off-by: Andrew Udvare <audvare@gmail.com>

1964 of 1964 branches covered (100.0%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 4 files covered. (100.0%)

7340 of 7340 relevant lines covered (100.0%)

5.0 hits per line