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

Issue 744913002: Restore order to the universe. (Closed)

Created:
6 years, 1 month ago by ghost stip (do not use)
Modified:
6 years, 1 month ago
Reviewers:
smut, friedman, navabi, smut
CC:
chromium-reviews, pgervais+watch_chromium.org, kjellander-cc_chromium.org, cmp-cc_chromium.org, stip+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/build.git@master
Project:
tools
Visibility:
Public.

Description

Restore order to the universe. R=friedman@chromium.org, navabi@google.com, smut@google.com, smut@chromium.org NOTRY=true NOTREECHECKS=True ^^^ tools/build is hosed due to pylint roll Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=293040

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M scripts/master/gatekeeper.py View 2 chunks +2 lines, -2 lines 0 comments Download
M scripts/master/goodrevisions.py View 1 chunk +1 line, -1 line 0 comments Download
M scripts/slave/gatekeeper_ng.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 19 (7 generated)
ghost stip (do not use)
ptal
6 years, 1 month ago (2014-11-20 19:06:27 UTC) #1
smut
lgtm
6 years, 1 month ago (2014-11-20 19:07:58 UTC) #3
navabi
lgtm
6 years, 1 month ago (2014-11-20 19:08:34 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/744913002/1
6 years, 1 month ago (2014-11-20 19:09:51 UTC) #6
friedman
On 2014/11/20 19:09:51, I haz the power (commit-bot) wrote: > CQ is trying da patch. ...
6 years, 1 month ago (2014-11-20 19:10:46 UTC) #7
friedman
On 2014/11/20 19:10:46, friedman wrote: > On 2014/11/20 19:09:51, I haz the power (commit-bot) wrote: ...
6 years, 1 month ago (2014-11-20 19:11:03 UTC) #8
commit-bot: I haz the power
Presubmit check for 744913002-1 failed and returned exit status 1. Running presubmit commit checks ...
6 years, 1 month ago (2014-11-20 19:11:33 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/744913002/1
6 years, 1 month ago (2014-11-20 19:12:54 UTC) #12
commit-bot: I haz the power
Presubmit check for 744913002-1 failed and returned exit status 1. Running presubmit commit checks ...
6 years, 1 month ago (2014-11-20 19:14:46 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/744913002/1
6 years, 1 month ago (2014-11-20 19:17:50 UTC) #16
commit-bot: I haz the power
Presubmit check for 744913002-1 failed and returned exit status 1. Running presubmit commit checks ...
6 years, 1 month ago (2014-11-20 19:19:34 UTC) #18
ghost stip (do not use)
6 years, 1 month ago (2014-11-20 19:30:38 UTC) #19
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as 293040.

Powered by Google App Engine
This is Rietveld 408576698