Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Groovy-8520 picocli-based CliBuilder #687

Closed
wants to merge 47 commits into from
Closed

Groovy-8520 picocli-based CliBuilder #687

wants to merge 47 commits into from

Commits on Apr 3, 2018

  1. GROOVY-8520 initial version of migrating CliBuilder from commons-cli … 

    …to picocli
    @remkop
    remkop committed Apr 3, 2018
    Configuration menu
    Copy the full SHA
    ecd3060 View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

    @remkop
    remkop committed Apr 3, 2018
    Configuration menu
    Copy the full SHA
    16645af View commit details
    Browse the repository at this point in the history

Commits on Apr 4, 2018

  1. GROOVY-8520 fix failing test CliBuilderTest.testConvertClass

    @remkop
    remkop committed Apr 4, 2018
    Configuration menu
    Copy the full SHA
    979d2f9 View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 fix failing test CliBuilderTest.testParseFromInstance: me… 

    …thod invocation parameters must be an array
    @remkop
    remkop committed Apr 4, 2018
    Configuration menu
    Copy the full SHA
    c42284a View commit details
    Browse the repository at this point in the history

  3. GROOVY-8520 cleanup: remove unused code (TypedOption not used any more)

    @remkop
    remkop committed Apr 4, 2018
    Configuration menu
    Copy the full SHA
    b9e5ca6 View commit details
    Browse the repository at this point in the history

  4. GROOVY-8520 for multi-value options, cli.optionName should return the… 

    … first element in the list
    @remkop
    remkop committed Apr 4, 2018
    Configuration menu
    Copy the full SHA
    261181a View commit details
    Browse the repository at this point in the history

  5. GROOVY-8520 fix failing CliBuilderTest.testMixedBurstingAndLongOption… 

    …s: to be backwards compatible, longOpt names should be registered with both a single hyphen and with two hyphens in picocli.
    @remkop
    remkop committed Apr 4, 2018
    Configuration menu
    Copy the full SHA
    69f2660 View commit details
    Browse the repository at this point in the history

  6. GROOVY-8520 fix CliBuilderTest.testMixedBurstingAndLongOptions: if yo… 

    …u reuse the same CliBuilder instance multiple times, the previous parse result interacts with the next parse result (e.g., boolean flags are toggled, collections are added to, etc)
    @remkop
    remkop committed Apr 4, 2018
    Configuration menu
    Copy the full SHA
    c405cfd View commit details
    Browse the repository at this point in the history

  7. GROOVY-8520 add back methods to OptionAccessor

    @remkop
    remkop committed Apr 4, 2018
    Configuration menu
    Copy the full SHA
    bc00052 View commit details
    Browse the repository at this point in the history

  8. GROOVY-8520 remove dependency on commons-cli from CliBuilderTest

    @remkop
    remkop committed Apr 4, 2018
    Configuration menu
    Copy the full SHA
    5ea2733 View commit details
    Browse the repository at this point in the history

Commits on Apr 5, 2018

  1. GROOVY-8520 bugfixes, revert TypedOption, support name property, supp… 

    …ress trace output
    @remkop
    remkop committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    7c1530e View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 expected output of usage help message is different with p… 

    …icocli
    @remkop
    remkop committed Apr 5, 2018
    Configuration menu
    Copy the full SHA
    29dab5f View commit details
    Browse the repository at this point in the history

Commits on Apr 6, 2018

  1. GROOVY-8520 replace Unmatched with PositionalParams to support strong… 

    …ly typed @unparsed fields/methods
    @remkop
    remkop committed Apr 6, 2018
    Configuration menu
    Copy the full SHA
    d0cf7a2 View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 Fix test: should not reuse CliBuilder instance for new input

    @remkop
    remkop committed Apr 6, 2018
    Configuration menu
    Copy the full SHA
    1821e98 View commit details
    Browse the repository at this point in the history

  3. Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

    @remkop
    remkop committed Apr 6, 2018
    Configuration menu
    Copy the full SHA
    7668333 View commit details
    Browse the repository at this point in the history

Commits on Apr 10, 2018

  1. GROOVY-8520 use new picocli-3.0.0-alpha-5 API, fix many bugs

    @remkop
    remkop committed Apr 10, 2018
    Configuration menu
    Copy the full SHA
    c342b2e View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 fix failing test

    @remkop
    remkop committed Apr 10, 2018
    Configuration menu
    Copy the full SHA
    e6652b9 View commit details
    Browse the repository at this point in the history

  3. GROOVY-8520 create new CliBuilder instance before parsing

    @remkop
    remkop committed Apr 10, 2018
    Configuration menu
    Copy the full SHA
    6a913e0 View commit details
    Browse the repository at this point in the history

  4. GROOVY-8520 create new CliBuilder instance before parsing

    @remkop
    remkop committed Apr 10, 2018
    Configuration menu
    Copy the full SHA
    699521b View commit details
    Browse the repository at this point in the history

  5. Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

    @remkop
    remkop committed Apr 10, 2018
    Configuration menu
    Copy the full SHA
    eb5aab2 View commit details
    Browse the repository at this point in the history

  6. GROOVY-8520 bump picocliVersion to v3.0.0-alpha-5

    @remkop
    remkop committed Apr 10, 2018
    Configuration menu
    Copy the full SHA
    98dbd91 View commit details
    Browse the repository at this point in the history

Commits on Apr 12, 2018

  1. GROOVY-8520 no need to create new CliBuilder instance before parsing … 

    …anymore
    @remkop
    remkop committed Apr 12, 2018
    Configuration menu
    Copy the full SHA
    29174a7 View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 no need to create new CliBuilder instance before parsing … 

    …anymore
    @remkop
    remkop committed Apr 12, 2018
    Configuration menu
    Copy the full SHA
    82a6f38 View commit details
    Browse the repository at this point in the history

  3. GROOVY-8520 no need to create new CliBuilder instance before parsing … 

    …anymore
    @remkop
    remkop committed Apr 12, 2018
    Configuration menu
    Copy the full SHA
    b22ce49 View commit details
    Browse the repository at this point in the history

  4. GROOVY-8520 bump picocliVersion to v3.0.0-alpha-6

    @remkop
    remkop committed Apr 12, 2018
    Configuration menu
    Copy the full SHA
    0d82aa9 View commit details
    Browse the repository at this point in the history

  5. Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

    @remkop
    remkop committed Apr 12, 2018
    Configuration menu
    Copy the full SHA
    251d319 View commit details
    Browse the repository at this point in the history

  6. GROOVY-8520 fix for testWithArgument

    @remkop
    remkop committed Apr 12, 2018
    Configuration menu
    Copy the full SHA
    55b5234 View commit details
    Browse the repository at this point in the history

Commits on Apr 13, 2018

  1. GROOVY-8520 docs; fix initialization bug on reuse (requires 3.0.0-bet… 

    …a-1 ArgSpec.Builder.hasInitialValue API)
    @remkop
    remkop committed Apr 13, 2018
    Configuration menu
    Copy the full SHA
    d1211ad View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 docs; fix NPE when usage() called before parse()

    @remkop
    remkop committed Apr 13, 2018
    Configuration menu
    Copy the full SHA
    eb12d2c View commit details
    Browse the repository at this point in the history

  3. GROOVY-8520 docs

    @remkop
    remkop committed Apr 13, 2018
    Configuration menu
    Copy the full SHA
    8317bcf View commit details
    Browse the repository at this point in the history

Commits on Apr 17, 2018

  1. GROOVY-8520 docs

    @remkop
    remkop committed Apr 17, 2018
    Configuration menu
    Copy the full SHA
    07e077f View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 add test

    @remkop
    remkop committed Apr 17, 2018
    Configuration menu
    Copy the full SHA
    698a7db View commit details
    Browse the repository at this point in the history

  3. Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

    @remkop
    remkop committed Apr 17, 2018
    Configuration menu
    Copy the full SHA
    057e40b View commit details
    Browse the repository at this point in the history

  4. GROOVY-8520 initial version in groovy-cli-picocli module

    @remkop
    remkop committed Apr 17, 2018
    Configuration menu
    Copy the full SHA
    e35a922 View commit details
    Browse the repository at this point in the history

  5. GROOVY-8520 revert back to commons-cli implementation (picocli versio… 

    …n lives in groovy-cli-picocli module)
    @remkop
    remkop committed Apr 17, 2018
    Configuration menu
    Copy the full SHA
    7238cee View commit details
    Browse the repository at this point in the history

  6. GROOVY-8520 add groovy-cli-picocli module

    @remkop
    remkop committed Apr 17, 2018
    Configuration menu
    Copy the full SHA
    8025626 View commit details
    Browse the repository at this point in the history

  7. GROOVY-8520 revert back to commons-cli implementation (picocli versio… 

    …n lives in groovy-cli-picocli module)
    @remkop
    remkop committed Apr 17, 2018
    Configuration menu
    Copy the full SHA
    6474256 View commit details
    Browse the repository at this point in the history

Commits on Apr 18, 2018

  1. GROOVY-8520 many fixes, use picocli-3.0.0-beta-1

    @remkop
    remkop committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    f958d1f View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

    @remkop
    remkop committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    f93f2ee View commit details
    Browse the repository at this point in the history

  3. GROOVY-8520 fix buggy test for spacing in usage message format

    @remkop
    remkop committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    3fc2a0c View commit details
    Browse the repository at this point in the history

Commits on Apr 19, 2018

  1. GROOVY-8520 when option is matched multiple times, return first typed… 

    … value, not the final value
    @remkop
    remkop committed Apr 19, 2018
    Configuration menu
    Copy the full SHA
    e868c78 View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 fix syntax nit

    @remkop
    remkop committed Apr 19, 2018
    Configuration menu
    Copy the full SHA
    b4da210 View commit details
    Browse the repository at this point in the history

  3. GROOVY-8520 better test method name

    @remkop
    remkop committed Apr 19, 2018
    Configuration menu
    Copy the full SHA
    a704be7 View commit details
    Browse the repository at this point in the history

Commits on Apr 20, 2018

  1. GROOVY-8520 documentation

    @remkop
    remkop committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    fe5b212 View commit details
    Browse the repository at this point in the history

  2. GROOVY-8520 fix for spacing in usage help message

    @remkop
    remkop committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    c9507a0 View commit details
    Browse the repository at this point in the history

  3. GROOVY-8520 small doc fixes

    @remkop
    remkop committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    3ab9662 View commit details
    Browse the repository at this point in the history

  4. GROOVY-8520 docs

    @remkop
    remkop committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    5e136ff View commit details
    Browse the repository at this point in the history