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

Unified Diff: testing/buildbot/chromium.linux.json

Issue 2903733004: (reland) Check missing headers in the build on CQ (Closed)
Patch Set: update whitelist Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/check_gn_headers_whitelist.txt ('k') | testing/scripts/check_gn_headers.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: testing/buildbot/chromium.linux.json
diff --git a/testing/buildbot/chromium.linux.json b/testing/buildbot/chromium.linux.json
index 27df774515d43a54d5684a6b3b6f5e6953c21591..9ed057d0308a5a708ddbce976e47f12ec179c62f 100644
--- a/testing/buildbot/chromium.linux.json
+++ b/testing/buildbot/chromium.linux.json
@@ -3519,11 +3519,23 @@
"Linux Builder": {
"additional_compile_targets": [
"all"
+ ],
+ "scripts": [
Dirk Pranke 2017/05/25 01:39:31 Why are you moving these from the "Linux Tests" bu
wychen 2017/05/25 02:05:24 Exactly. The analysis relies on having a fully com
+ {
+ "name": "check_gn_headers",
+ "script": "check_gn_headers.py"
+ }
]
},
"Linux Builder (dbg)": {
"additional_compile_targets": [
"all"
+ ],
+ "scripts": [
+ {
+ "name": "check_gn_headers",
+ "script": "check_gn_headers.py"
+ }
]
},
"Linux Tests": {
« no previous file with comments | « build/check_gn_headers_whitelist.txt ('k') | testing/scripts/check_gn_headers.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698