Tatsh / deltona / 30723844980 / 2

100%

master: 100%

push

github

project: resolve outstanding Ruff findings

Twenty findings were reported across the package, so a clean run gave no
signal about newly introduced problems.

Ignore S607 globally. Executables such as ffmpeg, killall, and taskkill
are deliberately resolved through PATH, which is the same reasoning that
already excludes S404 and S603, and it applies at eight call sites.

Suppress the deliberate gi.repository imports in the desktop command,
which stay lazy because pygobject is an optional dependency, and the
naive strptime call on a yt-dlp upload date, which has no timezone to
parse and only feeds a modification time.

Name the signal handler arguments in wineshell with a leading underscore
rather than suppressing the rule, since the handler must accept the two
arguments the signal module passes but uses neither.

Suppress the ambiguous character rule on the fullwidth test string,
which is the input the test exists to convert. The literal moves to its
own line so the suppression fits the line length.

Import is_binary_string at module scope. Ruff and yapf disagreed about
how to wrap the parenthesised form once a suppression comment made the
line longer, each undoing the other. The import needs no suppression at
module scope, binaryornot is a required dependency rather than an
optional one, and it was being re-imported on every iteration of a
nested loop. The tests patch it where it is now looked up.

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

438 of 438 branches covered (100.0%)

Branch coverage included in aggregate %.

3602 of 3602 relevant lines covered (100.0%)

1.0 hits per line