RFR: git-fork: add --sync option

Robin Westberg rwestberg at openjdk.java.net
Tue Dec 10 08:51:28 UTC 2019


On Tue, 10 Dec 2019 08:43:35 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hi all,
> 
> please review this patch that adds the `--sync` option to `git-fork`. Setting
> `--sync` (either via command-line or configuration file) makes `git-fork` sync
> the fork with the upstream repository as a last step. This ensures that the
> local clone of the fork on the user's machine is fully up to date with
> upstream.
> 
> Note that I hade to shuffle some code around in `git-sync` to allow for this,
> since `git-fork` simply just calls `git-sync` to implement this feaure.
> 
> Thanks,
> Erik
> 
> - [x] Manual testing of `git-fork`

Looks good, just one minor comment!

cli/src/main/java/org/openjdk/skara/cli/GitFork.java line 147:

> 146:                   .fullname("sync")
> 147:                   .helptext("Sync with the upstream repository after succesful fork")
> 148:                   .optional(),

Suggestion:

                  .helptext("Sync with the upstream repository after successful fork")

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

Marked as reviewed by rwestberg (Reviewer).

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


More information about the skara-dev mailing list