New issue
Advanced search Search tips

Issue 715856 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Task

Blocked on:
issue 770408
issue 710028



Sign in to add a comment

Simplify git-cl upload for gerrit

Project Member Reported by aga...@chromium.org, Apr 27 2017

Issue description

Ideas for simplification:
* Always set NOTIFY=NONE in refspec, always do a single SetReview call afterwards
  - this lets us do things like publish drafts upon upload
* Remove all R= lines from CL description
* Remove all CC= lines from description
* Probably more stuff which we should track here as we think of it
 
Status: Available (was: Untriaged)

Comment 2 by aga...@chromium.org, Apr 27 2017

Blockedon: 710028
These ideas are dependent on https://chromium-review.googlesource.com/c/479712/ so marking blocked on 710028.

Additional idea:
* Require that -s only be sent when there is an actual message to be sent (i.e. on subsequent uploads, require -m as well).

Comment 3 by aga...@chromium.org, Apr 27 2017

Labels: -Milestone-Afterglow Milestone-Launch
Owner: aga...@chromium.org
Status: Assigned (was: Available)
Blocking: 717197
Blocking: -717197
Labels: -Pri-2 -Milestone-Launch Milestone-Afterglow Pri-1
Not actually blocking. Would love to get to this, but it's not actually user-visible, and not a regression. Bumping to Afterglow, but marking Pri-1 so it gets done soon.
Blockedon: 770408
Labels: -Pri-1 Pri-2
Owner: ----
Status: Available (was: Assigned)
Labels: -Milestone-Afterglow
Removing Milestone-Afterglow, as it has ceased to have meaning. More refined milestones may be added back in the near future.
Labels: -Type-Bug -Proj-Gerrit-Migration Type-Task
Misleading_graph.pdf
0 bytes Download

Sign in to add a comment