TheKevJames / coveralls-python / 2134ea74-f00c-471b-abbd-8c773d1e63e6

95%

master: 96%

Build Type

Pull #756

circleci

Committed by TheKevJames

Commit Message

refactor(cli): consistent setting names across every interface

Give each setting one spelling across all interfaces. Rename coveralls_host->
host and config_file->rcfile, expose host/skip_ssl_verify/parallel as CLI
flags, and define the CLI once via shared Annotated option specs with debug as
a thin wrapper over a shared dispatch. Old CLI flags and config keys keep
working as deprecated aliases that emit a warning.

changelog:
Every setting now uses one consistent name across all interfaces: a setting
``foo_bar`` is ``--foo-bar`` on the CLI, ``COVERALLS_FOO_BAR`` as an environment
variable, and ``foo_bar`` in ``.coveralls.yml`` or as a ``Coveralls()`` keyword.
``--host``, ``--skip-ssl-verify`` and ``--parallel`` are now available as CLI
flags. The config keys ``coveralls_host`` and ``config_file`` were renamed to
``host`` and ``rcfile``, and the CLI flags ``--service``, ``--basedir`` and
``--srcdir`` to ``--service-name``, ``--base-dir`` and ``--src-dir``. The old
spellings still work but emit a deprecation warning and will be removed in a
future release.

Pull Request Pull Request #756: refactor(cli): consistent setting names across every interface

Coverage Stats

144 of 154 branches covered (93.51%)

Branch coverage included in aggregate %.

52 of 55 new or added lines in 3 files covered. (94.55%)

1 existing line in 1 file now uncovered.

489 of 516 relevant lines covered (94.77%)

5.69 hits per line