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

Issue 83173008: CQ: don't treat the commit box as unchecked on network errors (Closed)

Created:
7 years, 1 month ago by Paweł Hajdan Jr.
Modified:
7 years ago
Reviewers:
Isaac (away)
CC:
chromium-reviews, cmp-cc_chromium.org, ilevy-cc_chromium.org
Visibility:
Public.

Description

CQ: don't treat the commit box as unchecked on network errors BUG=309076 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=237402

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -16 lines) Patch
M commit-queue/pending_manager.py View 2 chunks +10 lines, -16 lines 1 comment Download

Messages

Total messages: 5 (0 generated)
Paweł Hajdan Jr.
Please review. Just in case you ask, previous review was https://codereview.chromium.org/26951002 This is working fine ...
7 years, 1 month ago (2013-11-22 18:01:10 UTC) #1
Isaac (away)
lgtm but please add a bug per chase's request in the other CL
7 years ago (2013-11-26 01:01:42 UTC) #2
Paweł Hajdan Jr.
On 2013/11/26 01:01:42, Isaac wrote: > lgtm but please add a bug per chase's request ...
7 years ago (2013-11-26 20:36:59 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/phajdan.jr@chromium.org/83173008/1
7 years ago (2013-11-26 20:37:05 UTC) #4
commit-bot: I haz the power
7 years ago (2013-11-26 20:37:23 UTC) #5
Message was sent while issue was closed.
Change committed as 237402

Powered by Google App Engine
This is Rietveld 408576698