TheKevJames / coveralls-python / 8373510b-b122-4d86-b127-2d01d6d3b689

95%

master: 97%

push

circleci

fix(cli): keep shared options on each subcommand, not the callback

The previous commit moved shared modifiers onto the Typer callback and read
them via the context, which forced options to precede the subcommand name.
That silently broke the long-standing 'coveralls debug --rcfile=x' form
(options after the subcommand).

Scope each command's options directly on the command instead, exactly as the
debug subcommand did before, so options are still given after the command
name. The only removal is the verb flags (--output/--submit/--finish), which
debug should never have accepted; those remain as hidden deprecated aliases on
the default command for backwards compatibility. finish/upload intentionally
do not advertise the report-building modifiers they ignore, so passing them
is a clean usage error.

153 of 164 branches covered (93.29%)

Branch coverage included in aggregate %.

30 of 30 new or added lines in 1 file covered. (100.0%)

539 of 567 relevant lines covered (95.06%)

5.7 hits per line