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

Issue 2044663003: git cl try: clarify where the default set of bots is coming from. (Closed)

Created:
4 years, 6 months ago by tandrii(chromium)
Modified:
4 years, 6 months ago
CC:
chromium-reviews, tandrii+omg_git_cl_chromium.org, dpranke+depot_tools_chromium.org, iannucci+depot_tools_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M git_cl.py View 1 chunk +3 lines, -0 lines 1 comment Download

Messages

Total messages: 8 (2 generated)
tandrii(chromium)
PTAL example output for this CL in depot_tools: $ git cl try Loaded default bots ...
4 years, 6 months ago (2016-06-07 14:53:28 UTC) #1
dsinclair
lgtm
4 years, 6 months ago (2016-06-07 14:55:04 UTC) #2
Sergiy Byelozyorov
lgtm w/ optional comment https://codereview.chromium.org/2044663003/diff/1/git_cl.py File git_cl.py (right): https://codereview.chromium.org/2044663003/diff/1/git_cl.py#newcode4404 git_cl.py:4404: print('CQ config exists (%s) but ...
4 years, 6 months ago (2016-06-07 14:58:14 UTC) #3
tandrii(chromium)
On 2016/06/07 14:58:14, Sergiy Byelozyorov wrote: > lgtm w/ optional comment > > https://codereview.chromium.org/2044663003/diff/1/git_cl.py > ...
4 years, 6 months ago (2016-06-07 14:59:48 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2044663003/1
4 years, 6 months ago (2016-06-07 15:00:02 UTC) #6
commit-bot: I haz the power
4 years, 6 months ago (2016-06-07 15:03:13 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/depot_tools/+/b93dd2b2f4d87f...

Powered by Google App Engine
This is Rietveld 408576698