RFR: git-jcheck: add pre-push support

Robin Westberg rwestberg at openjdk.java.net
Mon Feb 24 14:29:44 UTC 2020


On Mon, 24 Feb 2020 14:21:09 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hi all,
> 
> please review this patch that adds support for the `--pre-push` and
> `--setup-pre-push-hook` to `git-jcheck`. Setting up a pre-push hook can
> beneficial for OpenJDK projects where contributors push directly (where they
> don't use `git-pr` and pull requests).
> 
> Testing:
> - Manual testing on Linux x64
> 
> Thanks,
> Erik

Looks good, just be consistent with the dash. :)

cli/src/main/java/org/openjdk/skara/cli/GitJCheck.java line 119:

> 118:                   .fullname("setup-pre-push-hook")
> 119:                   .helptext("Set up a pre push hook that runs jcheck")
> 120:                   .optional(),

Suggestion:

                  .helptext("Set up a pre-push hook that runs jcheck")

-------------

Marked as reviewed by rwestberg (Reviewer).

PR: https://git.openjdk.java.net/skara/pull/470


More information about the skara-dev mailing list