TheKevJames / coveralls-python / a848812e-efa0-4261-b42e-0e8e0394c12a

95%

master: 96%

Pull #759

circleci

feat(cli): promote finish/save/upload verbs to subcommands

The CLI mixed two kinds of thing in one flat namespace: verbs (mutually
exclusive modes selected by --finish/--output/--submit) and modifiers that
tune a verb. Modelling verbs as independent flags left no guardrails -- e.g.
the debug subcommand inherited the full flag set and would silently ignore
--output/--submit/--finish, and conflicting verbs resolved by an undocumented
precedence ladder rather than erroring.

Promote the verbs to first-class subcommands alongside the existing debug:

  coveralls              collect + submit (unchanged bare invocation)
  coveralls finish       parallel-finish webhook       (was --finish)
  coveralls save FILE    write report to a file        (was --output FILE)
  coveralls upload FILE  upload a pre-generated report (was --submit FILE)
  coveralls debug        build report, dump to stdout, don't send

Each command declares only the options it uses, so options are still given
after the command name (e.g. `coveralls debug --rcfile=x`) and inapplicable
flag/verb combinations become clean usage errors instead of silent no-ops.

Backwards compatibility: --finish, --output and --submit remain as hidden,
deprecated aliases on the default command that dispatch to the new subcommands
and emit a deprecation warning in the same style as the existing
rcfile/config_file and --service/--service-name deprecations. These aliases
dispatch identically to the old flat CLI, including applying --merge before
the action (a no-op for submit/finish payloads, but preserved for fidelity).
All the modifiers the old flat namespace accepted alongside --finish/--submit
(--service-name/--rcfile/--base-dir/--src-dir/...) are still accepted on the
default command, so existing pipelines are unaffected. Passing a deprecated
verb flag together with a subcommand, or more than one verb flag at once, is
now rejected rather than silently resolved. Usage errors are rendered as a
message plus exi... (continued)

156 of 166 branches covered (93.98%)

Branch coverage included in aggregate %.

95 of 96 new or added lines in 1 file covered. (98.96%)

541 of 568 relevant lines covered (95.25%)

5.71 hits per line