diff options
| author | Matt T. Proud <matt.proud@gmail.com> | 2025-07-04 20:03:02 +0200 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2025-07-09 08:35:44 -0700 |
| commit | 48aa97b42827239f9fd02d1b5ad75adc200c7a72 (patch) | |
| tree | 60f37a646805ad81076b21ed4e0b1281dcc4524b /git-codereview/submit.go | |
| parent | c4af2b031e13e282462ea8ae4311bfabe115ae9d (diff) | |
| download | go-x-review-1.16.0.tar.xz | |
git-codereview: frame sync as git codereview syncv1.16.0
The codereview binary referred to the sync subcommand as merely "git
sync" in a number of places in user surfaces. This was wrong and
misleading, so the code now refers to it with the codereview parent
subcommand in the name (i.e., a "git codereview sync").
Change-Id: Icf3f96a8c76b57319bc3101b3e9fd0cbd8cecea9
Reviewed-on: https://go-review.googlesource.com/c/review/+/685855
Auto-Submit: Sean Liao <sean@liao.dev>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Reviewed-by: Sean Liao <sean@liao.dev>
Reviewed-by: David Chase <drchase@google.com>
Diffstat (limited to 'git-codereview/submit.go')
| -rw-r--r-- | git-codereview/submit.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/git-codereview/submit.go b/git-codereview/submit.go index ce629c3..6cfb809 100644 --- a/git-codereview/submit.go +++ b/git-codereview/submit.go @@ -59,7 +59,7 @@ func cmdSubmit(args []string) { } // Sync client to revision that Gerrit committed, but only if we can do it cleanly. - // Otherwise require user to run 'git sync' themselves (if they care). + // Otherwise require user to run 'git codereview sync' themselves (if they care). run("git", "fetch", "-q") if len(cs) == 1 && len(b.Pending()) == 1 { if err := runErr("git", "checkout", "-q", "-B", b.Name, g.CurrentRevision, "--"); err != nil { @@ -166,7 +166,7 @@ func submitCheck(g *GerritChange) error { // Not yet "MERGED", so try the submit. // "SUBMITTED" is a weird state. It means that Submit has been clicked once, // but it hasn't happened yet, usually because of a merge failure. - // The user may have done git sync and may now have a mergable + // The user may have done git codereview sync and may now have a mergable // copy waiting to be uploaded, so continue on as if it were "NEW". case "MERGED": |