sqitchers / sqitch / 338

91%

develop: 100%

push

travis-ci-com

Merge branch 'option-unification'

This mege changes the way Sqitch handles command-line argument parsing to
allow both core and command options to appear either before or after the
command. Previously, core options had to come before the command name, and
command options after. No more. The caveat is that command options that take
arguments should either appear after the command or use the `--opt=val` syntax
instead of `--opt val`, so that Sqitch doesn't think `val` is the command.

Even in that case, it will keep searching the rest of the arguments to find a
valid command. However, to minimize this challenge, the documentation now
suggests and demonstrates putting all options after the command, like so:

    sqitch [command] [options]

With the ability to mix and match core and command options, in order to
eliminate conflicts between the two categories of options, this merge also
moves all non-global options from the core to the commands that need them
(closes #250). Only options that affect output and runtime context remain in
the core:

* -C --chdir --cd <dir>  Change to directory before performing any actions
*    --etc-path          Print the path to the etc directory and exit
*    --no-pager          Do not pipe output into a pager
*    --quiet             Quiet mode with non-error output suppressed
* -V --verbose           Increment verbosity
*    --version           Print the version number and exit
*    --help              Show a list of commands and exit
*    --man               Print the introductory documentation and exit

Note that single-character aliases are now uppercase for core options (`-v` is
still supported for `--verbose`, but no longer documented). The options that
were moved include:

-   plan-file|f=s
-   engine=s
-   registry=s
-   client|db-client=s
-   db-name|d=s
-   db-username|db-user|u=s
-   db-host|h=s
-   db-port|p=i
-   top-dir|dir=s

I believe all the commands that need to use one or more of these options now
suppo... (continued)

72 of 72 new or added lines in 8 files covered. (100.0%)

4133 of 4413 relevant lines covered (93.66%)

62.24 hits per line