[GitHub] groovy pull request #:

2018-07-16 Thread remkop
Github user remkop commented on the pull request: https://github.com/apache/groovy/commit/a798866fcfd98f7b2cd46e948b37217a8349f609#commitcomment-29728904 Thanks for the heads up! The test fails because of a change in the error message format. I’ll fix it tonight and I’ll make

[GitHub] groovy pull request #770: GROOVY-8690 bump picocli to v3.2.0 from 3.1.0

2018-07-06 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/770 GROOVY-8690 bump picocli to v3.2.0 from 3.1.0 Details in https://issues.apache.org/jira/browse/GROOVY-8690 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] groovy pull request #745: GROOVY-8621 Grape command line tool synopsis impro...

2018-06-02 Thread remkop
Github user remkop closed the pull request at: https://github.com/apache/groovy/pull/745 ---

[GitHub] groovy pull request #745: GROOVY-8621 Grape command line tool synopsis impro...

2018-06-01 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/745 GROOVY-8621 Grape command line tool synopsis improvement version2 see https://issues.apache.org/jira/browse/GROOVY-8621 Previously merged PR had layout issues. Fixed in this PR. You can

[GitHub] groovy pull request #743: GROOVY-8621 Grape command line tool synopsis impro...

2018-06-01 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/743 GROOVY-8621 Grape command line tool synopsis improvement see https://issues.apache.org/jira/browse/GROOVY-8621 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] groovy pull request #742: GROOVY-8620 Update docs for Grape dependency manag...

2018-06-01 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/742 GROOVY-8620 Update docs for Grape dependency manager see https://issues.apache.org/jira/browse/GROOVY-8620 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] groovy pull request #741: GROOVY-8618 Update docs for GroovyConsole

2018-06-01 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/741 GROOVY-8618 Update docs for GroovyConsole see https://issues.apache.org/jira/browse/GROOVY-8618 You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] groovy pull request #740: GROOVY-8619 Update docs for Groovy Shell

2018-06-01 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/740 GROOVY-8619 Update docs for Groovy Shell see https://issues.apache.org/jira/browse/GROOVY-8619 You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] groovy pull request #728: GROOVY-8605 Autocompletion for Groovy command line...

2018-05-26 Thread remkop
Github user remkop closed the pull request at: https://github.com/apache/groovy/pull/728 ---

[GitHub] groovy pull request #728: GROOVY-8605 Autocompletion for Groovy command line...

2018-05-26 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/728 GROOVY-8605 Autocompletion for Groovy command line tools This PR adds bash completion scripts for the Groovy command line tools in {{installdir/bin}}. This will enhance the user experience for users

[GitHub] groovy pull request #711: GROOVY-8589 bugfix: should not show error about mi...

2018-05-18 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/711 GROOVY-8589 bugfix: should not show error about missing mandatory args when --help or --version is specified GROOVY-8589 bugfix: should not show error about missing mandatory args when --help or

[GitHub] groovy pull request #710: GROOVY-8588 use lowercase -v for --version in groo...

2018-05-18 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/710 GROOVY-8588 use lowercase -v for --version in groovyc to be consistent with other Groovy command line tools GROOVY-8588 use lowercase -v for --version in groovyc to be consistent with other Groovy

[GitHub] groovy pull request #704: GROOVY-8577 Migrate org.codehaus.groovy.tools.Grap...

2018-05-14 Thread remkop
Github user remkop commented on a diff in the pull request: https://github.com/apache/groovy/pull/704#discussion_r188142573 --- Diff: src/main/groovy/org/codehaus/groovy/tools/GrapeMain.groovy --- @@ -19,290 +19,323 @@ package org.codehaus.groovy.tools import

[GitHub] groovy pull request #704: GROOVY-8577 Migrate org.codehaus.groovy.tools.Grap...

2018-05-11 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/704 GROOVY-8577 Migrate org.codehaus.groovy.tools.GrapeMain.groovy to picocli GROOVY-8577 Migrate org.codehaus.groovy.tools.GrapeMain.groovy to picocli You can merge this pull request into a Git

[GitHub] groovy pull request #700: GROOVY-8576 Remove Java2GroovyMain dependency on c...

2018-05-10 Thread remkop
Github user remkop closed the pull request at: https://github.com/apache/groovy/pull/700 ---

[GitHub] groovy pull request #694: GROOVY-8567 Migrate Groovyc to picocli

2018-05-10 Thread remkop
Github user remkop closed the pull request at: https://github.com/apache/groovy/pull/694 ---

[GitHub] groovy pull request #703: GROOVY-8567 Migrate Groovyc to picocli

2018-05-10 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/703 GROOVY-8567 Migrate Groovyc to picocli Migrate groovyc to picocli. Rebased and squashed commits. This PR replaces #694. You can merge this pull request into a Git repository by running

[GitHub] groovy pull request #697: GROOVY-8569 GroovyMain picocli migration

2018-05-10 Thread remkop
Github user remkop closed the pull request at: https://github.com/apache/groovy/pull/697 ---

[GitHub] groovy pull request #702: GROOVY-8569 GroovyMain picocli migration

2018-05-10 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/702 GROOVY-8569 GroovyMain picocli migration Migrate `groovy.ui.GroovyMain` from commons-cli to picocli. Replaces PR #697 You can merge this pull request into a Git repository by running

[GitHub] groovy pull request #696: GROOVY-8520 fixes (attempt 2)

2018-05-10 Thread remkop
Github user remkop closed the pull request at: https://github.com/apache/groovy/pull/696 ---

[GitHub] groovy pull request #701: GROOVY-8520 fixes (attempt 3)

2018-05-10 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/701 GROOVY-8520 fixes (attempt 3) Rebased on master, replaces PR #696 You can merge this pull request into a Git repository by running: $ git pull https://github.com/remkop/groovy GROOVY-8520-fixes

[GitHub] groovy pull request #700: GROOVY-8576 Remove Java2GroovyMain dependency on c...

2018-05-10 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/700 GROOVY-8576 Remove Java2GroovyMain dependency on commons-cli `org.codehaus.groovy.antlr.java.Java2GroovyMain` can be simplified by removing its dependency on commons-cli: it does not have any

[GitHub] groovy pull request #697: GROOVY-8569 GroovyMain picocli migration

2018-05-06 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/697 GROOVY-8569 GroovyMain picocli migration Migrate `groovy.ui.GroovyMain` from commons-cli to picocli. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] groovy pull request #696: GROOVY-8520 fixes (attempt 2)

2018-05-06 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/696 GROOVY-8520 fixes (attempt 2) * add CliBuilder.setParser and setFormatter methods that ignore the specified values and print a warning to stderr to easy the transition for existing applications

[GitHub] groovy pull request #695: GROOVY-8520 fixes

2018-05-06 Thread remkop
Github user remkop closed the pull request at: https://github.com/apache/groovy/pull/695 ---

[GitHub] groovy pull request #695: GROOVY-8520 fixes

2018-05-06 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/695 GROOVY-8520 fixes * add CliBuilder.setParser and setFormatter methods that ignore the specified values and print a warning to stderr to easy the transition for existing applications * bugfix

[GitHub] groovy pull request #694: GROOVY-8567 Migrate Groovyc to picocli

2018-05-06 Thread remkop
Github user remkop commented on a diff in the pull request: https://github.com/apache/groovy/pull/694#discussion_r186292414 --- Diff: src/main/java/org/codehaus/groovy/tools/FileSystemCompiler.java --- @@ -77,18 +73,6 @@ public void compile(File[] files) throws Exception

[GitHub] groovy pull request #694: GROOVY-8567 Migrate Groovyc to picocli

2018-05-04 Thread remkop
Github user remkop commented on a diff in the pull request: https://github.com/apache/groovy/pull/694#discussion_r186035996 --- Diff: src/main/java/org/codehaus/groovy/tools/FileSystemCompiler.java --- @@ -77,18 +73,6 @@ public void compile(File[] files) throws Exception

[GitHub] groovy pull request #694: GROOVY-8567 Migrate Groovyc to picocli

2018-05-04 Thread remkop
Github user remkop commented on a diff in the pull request: https://github.com/apache/groovy/pull/694#discussion_r186035195 --- Diff: src/main/java/org/codehaus/groovy/tools/FileSystemCompiler.java --- @@ -77,18 +73,6 @@ public void compile(File[] files) throws Exception

[GitHub] groovy pull request #694: GROOVY-8567 Migrate Groovyc to picocli

2018-05-04 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/694 GROOVY-8567 Migrate Groovyc to picocli GROOVY-8567 Migrate Groovyc to picocli See comments in the JIRA ticket. You can merge this pull request into a Git repository by running: $ git

[GitHub] groovy pull request #692: GROOVY-8520 CliBuilder groovydoc improvements

2018-04-29 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/692 GROOVY-8520 CliBuilder groovydoc improvements Groovy-doc improvements for CliBuilder. You can merge this pull request into a Git repository by running: $ git pull https://github.com/remkop

[GitHub] groovy pull request #687: Groovy-8520 picocli-based CliBuilder

2018-04-21 Thread remkop
Github user remkop closed the pull request at: https://github.com/apache/groovy/pull/687 ---

[GitHub] groovy pull request #688: GROOVY-8520 add picocli-based CliBuilder

2018-04-21 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/688 GROOVY-8520 add picocli-based CliBuilder Rebased on the GROOVY_2_5_X branch with all commits squashed. Requires picocli 3.0.0-beta-2. Renamed package to {{groovy.cli.picocli

[GitHub] groovy pull request #687: Groovy-8520 picocli-based CliBuilder

2018-04-18 Thread remkop
GitHub user remkop opened a pull request: https://github.com/apache/groovy/pull/687 Groovy-8520 picocli-based CliBuilder Details are in the JIRA: https://issues.apache.org/jira/browse/GROOVY-8520 You can merge this pull request into a Git repository by running: $ git