| Issue 2229: | Gerrit review update should also return the URL | |
| 2 people starred this issue and may be notified of changes. | Back to list |
As discussed with Shawn on the EclipseCon Europe it would be helpful IMHO if Gerrit returns also the URL for an updated Gerrit review. Currently Gerrit returns the URL only during the initial creation. This makes finding the updated Gerrit review a little bit more complex than necessary.
Oct 31, 2013
Project Member
#1
edwin.ke...@gmail.com
Oct 31, 2013
I push via Eclipse. First push returns the following: Repository ssh://lvogel@git.eclipse.org:29418/platform/eclipse.platform.ui.git Processing changes: new: 1, refs: 1 Processing changes: new: 1, refs: 1, done ---------- Reviewing commit: commit 0b0c70e59dbeb2a8905978e246ceaaebefe05206 1383224863 ----sp Authored by: Lars Vogel <Lars.Vogel@gmail.com> The author is a committer on the project. This commit passes Eclipse validation. New Changes: https://git.eclipse.org/r/17921 The update returns the following: Repository ssh://lvogel@git.eclipse.org:29418/platform/eclipse.platform.ui.git Processing changes: updated: 1, refs: 1 Processing changes: updated: 1, refs: 1, done ---------- Reviewing commit: commit 87d89f3cb08c0f6115832b0ee52e18a2ba192758 1383224898 ----sp Authored by: Lars Vogel <Lars.Vogel@gmail.com> The author is a committer on the project. This commit passes Eclipse validation. (W) 87d89f3: no files changed, message updated ----------- Would be great if the second answer also include in this example the URL: https://git.eclipse.org/r/17921
Oct 31, 2013
(No comment was entered for this change.)
Summary:
Gerrit review update should also return the URL
(was: Gerrit review update should also return the ULR)
Oct 31, 2013
Thanks Edwin, via Twitter the Eclipse community cheered with happiness after your uploaded review. :-)
Nov 5, 2013
(No comment was entered for this change.)
Status:
Submitted
Labels: FixedIn-2.9
May 20, 2014
Issue 2671 has been merged into this issue.
Jul 18, 2014
(No comment was entered for this change.)
Status:
Released
|
|
| ► Sign in to add a comment |