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

Issue 444073002: Adds two more analyze exclusions (Closed)

Created:
6 years, 4 months ago by sky
Modified:
6 years, 4 months ago
Reviewers:
tonyg, Nico
CC:
chromium-reviews
Project:
chromium
Visibility:
Public.

Description

Adds two more analyze exclusions build/android/pylib: telemetry build/get_landmines.py: needs to force build BUG=399774, 399710 TEST=none R=tonyg@chromium.org, thakis@chromium.org TBR=thakis@chromium.org, tonyg@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287827

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M testing/buildbot/chromium_trybot.json View 2 chunks +3 lines, -3 lines 1 comment Download

Messages

Total messages: 5 (0 generated)
sky
TBR since trivial change. https://codereview.chromium.org/444073002/diff/1/testing/buildbot/chromium_trybot.json File testing/buildbot/chromium_trybot.json (left): https://codereview.chromium.org/444073002/diff/1/testing/buildbot/chromium_trybot.json#oldcode214 testing/buildbot/chromium_trybot.json:214: "filter_builders": [ filter_builders has been ...
6 years, 4 months ago (2014-08-06 18:28:02 UTC) #1
sky
The CQ bit was checked by sky@chromium.org
6 years, 4 months ago (2014-08-06 18:28:05 UTC) #2
tonyg
lgtm
6 years, 4 months ago (2014-08-06 18:29:20 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sky@chromium.org/444073002/1
6 years, 4 months ago (2014-08-06 18:29:27 UTC) #4
sky
6 years, 4 months ago (2014-08-06 19:25:52 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as 287827 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698