New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 750863 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 739928
Owner: ----
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: ----


Previous locations:
gerrit:6901


Sign in to add a comment

Consider using amended commit messages as new gerrit change description

Project Member Reported by mcchou@google.com, Jul 31 2017

Issue description

CL: https://chromium-review.googlesource.com/c/580314

Expected behavior:
After submitting a new patch set, the changes on the commit message should be reflected on both the patch summary and the commit message file.

Actual behavior:
After submitting a new patch set, the changes on the commit message were neither showed up in the patch set summary nor the commit message file.
 

Comment 1 by mcchou@chromium.org, Jul 31 2017

Cc: mcchou@google.com
Labels: Priority-1
Owner: logan@google.com

Comment 2 by logan@google.com, Jul 31 2017

Status: WontFix (was: New)
Once you've created a code review with git cl upload, you have to use either git cl description or the web UI to modify the commit message. If you only use git commit --amend, your local commit message edit will be ignored.

Comment 3 by mcchou@chromium.org, Jul 31 2017

Re #2,

Although this is the current workflow, but this seems to be a worthy one to fix, so that extra step is not needed just to update the commit message.

Comment 4 by logan@google.com, Jul 31 2017

Components: -PolyGerrit
Labels: -Priority-1 Proj-Gerrit-Migration
Owner: ----
Status: New (was: WontFix)

Comment 5 by logan@google.com, Jul 31 2017

Project: chromium
Moved issue gerrit:6901 to now be  issue chromium:750863 .

Comment 6 by logan@google.com, Jul 31 2017

Components:
Status: (was: WontFix)

Comment 7 by logan@google.com, Jul 31 2017

Owner: ----
Components: Infra>Codereview>Gerrit

Comment 9 by aga...@chromium.org, Aug 28 2017

Cc: aga...@chromium.org
Components: Infra>SDK
Labels: Milestone-Afterglow Pri-3
Status: Available
Summary: Consider using amended commit messages as new gerrit change description (was: The changes on commit message are not reflected on Chromium patch set in Gerrit)
Yes, we recognize that this behavior is confusing to people who are used to Gerrit. It is, however, the same behavior as Rietveld has always had.

We would like to consider changing this, but it will have to be carefully considered.
Labels: -Milestone-Afterglow
Removing Milestone-Afterglow, as it has ceased to have meaning. More refined milestones may be added back in the near future.
Mergedinto: 739928
Status: Duplicate (was: Available)

Sign in to add a comment