Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(286)

Issue 1868313002: git cl upload: commit subject is default patchset title (Closed)

Created:
4 years, 8 months ago by nodir
Modified:
4 years, 6 months ago
CC:
chromium-reviews, dpranke+depot_tools_chromium.org, iannucci+depot_tools_chromium.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

git cl upload: commit subject is default patchset title git cl upload asks the user to provide a patchset title. If the user makes a commit per patchset, the subject of the commit may be exactly what patchset title should be. R=dnj@chromium.org, vadimsh@chromium.org BUG=582419 Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/7aafc3f4adca9d61017a95b9a8e53e7c4ac4ce4a

Patch Set 1 #

Patch Set 2 : move strip #

Total comments: 2

Patch Set 3 : s/Current/MostRecent/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -9 lines) Patch
M third_party/upload.py View 1 2 6 chunks +17 lines, -9 lines 0 comments Download

Messages

Total messages: 21 (9 generated)
nodir
PTAL
4 years, 8 months ago (2016-04-08 20:28:31 UTC) #4
tandrii(chromium)
drive-by review from tired git cl hacker: can you add a test?
4 years, 8 months ago (2016-04-09 05:34:14 UTC) #7
agable
LGTM % nit https://codereview.chromium.org/1868313002/diff/40001/third_party/upload.py File third_party/upload.py (right): https://codereview.chromium.org/1868313002/diff/40001/third_party/upload.py#newcode1048 third_party/upload.py:1048: def GetCurrentCommitSummary(self): nit: s/Current/MostRecent/ feel free ...
4 years, 6 months ago (2016-06-09 22:15:14 UTC) #9
nodir
sorry tandrii. I've tested it locally. It works, I promise https://codereview.chromium.org/1868313002/diff/40001/third_party/upload.py File third_party/upload.py (right): https://codereview.chromium.org/1868313002/diff/40001/third_party/upload.py#newcode1048 ...
4 years, 6 months ago (2016-06-09 22:31:34 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1868313002/60001
4 years, 6 months ago (2016-06-09 22:31:42 UTC) #13
commit-bot: I haz the power
Committed patchset #3 (id:60001) as https://chromium.googlesource.com/chromium/tools/depot_tools/+/7aafc3f4adca9d61017a95b9a8e53e7c4ac4ce4a
4 years, 6 months ago (2016-06-09 22:35:00 UTC) #15
tandrii(chromium)
oh, this is upload, yeah don't add a test. With some luck, this code will ...
4 years, 6 months ago (2016-06-10 10:45:15 UTC) #16
nodir
On 2016/06/10 10:45:15, tandrii(chromium) wrote: > oh, this is upload, yeah don't add a test. ...
4 years, 6 months ago (2016-06-10 16:07:31 UTC) #17
nodir
On 2016/06/10 16:07:31, nodir wrote: > On 2016/06/10 10:45:15, tandrii(chromium) wrote: > > oh, this ...
4 years, 6 months ago (2016-06-10 16:09:03 UTC) #18
tandrii(chromium)
On 2016/06/10 16:09:03, nodir wrote: > On 2016/06/10 16:07:31, nodir wrote: > > On 2016/06/10 ...
4 years, 6 months ago (2016-06-10 16:20:20 UTC) #19
qyearsley
Thanks Nodir! I just noticed this change while uploading a patchset. Having to re-input the ...
4 years, 6 months ago (2016-06-14 00:02:46 UTC) #20
nodir
4 years, 6 months ago (2016-06-14 00:20:15 UTC) #21
Message was sent while issue was closed.
On 2016/06/14 00:02:46, qyearsley wrote:
> Thanks Nodir! I just noticed this change while uploading a patchset. Having to
> re-input the patchset title which is the same as a commit message I just added
> was one of those little things that I never noticed annoying me until it was
> fixed :-D

You are welcome

Powered by Google App Engine
This is Rietveld 408576698