• 
      

    Allow the common rb-site options to be used in subcommands.

    Review Request #11491 — Created Feb. 26, 2021 and submitted

    Information

    Review Board
    release-4.0.x

    Reviewers

    When moving to argparse, the common set of options (--noinput,
    --debug, etc.) were no longer able to be provided after the
    subcommand. That is, rb-site install --noinput no longer worked.

    This change addresses that by moving those options into a common
    argparser and setting that as a "parent" for all others. This allows
    them to all share the same set of options, allowing placement either
    before or after the subcommand.

    There's also a small tweak to one of the options, which had a wrong
    description.

    Tested rb-site install and rb-site upgrade with common options
    both before and after the subcommand, and verified they worked in
    both cases.

    Summary ID
    Allow the common rb-site options to be used in subcommands.
    When moving to `argparse`, the common set of options (`--noinput`, `--debug`, etc.) were no longer able to be provided after the subcommand. That is, `rb-site install --noinput` no longer worked. This change addresses that by moving those options into a common argparser and setting that as a "parent" for all others. This allows them to all share the same set of options, allowing placement either before or after the subcommand. There's also a small tweak to one of the options, which had a wrong description.
    2df96e02d609aff7866d15d7eca02b169ffdc618
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-4.0.x (4b88ecc)