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

Unified Diff: remoting/PRESUBMIT.py

Issue 1166123004: Add simple presubmit check to remoting. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/PRESUBMIT.py
diff --git a/remoting/PRESUBMIT.py b/remoting/PRESUBMIT.py
new file mode 100644
index 0000000000000000000000000000000000000000..5c69f6acc3fe27f263bc106649bc1ea8a7a5ccdd
--- /dev/null
+++ b/remoting/PRESUBMIT.py
@@ -0,0 +1,25 @@
+# Copyright 2015 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+"""Presubmit script for remoting.
+
+See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
+for more details about the presubmit API built into depot_tools.
+"""
+
+BROWSER_TEST_INSTRUCTIONS_LINK = (
+ "https://wiki.corp.google.com/twiki/bin/view/Main/ChromotingWaterfall#Running_on_a_Swarming_bot.")
kelvinp 2015/06/08 22:11:01 This link won't be accessible for non-googlers. B
+
+
+def CheckChangeOnUpload(input_api, output_api):
+ print "*******IMPORTANT NOTE*******"
+ print "Before committing, please run Remoting browser_tests."
+ print "Instructions: %s" % BROWSER_TEST_INSTRUCTIONS_LINK
+ print "Make sure all tests pass."
+ return []
+
+def CheckChangeOnCommit(input_api, output_api):
+ """TODO(anandc): Run browser-tests on the Chromoting waterfall as part of
Jamie 2015/06/08 22:46:28 Please create a bug tracking this work and referen
anandc 2015/06/08 23:40:09 Done.
+ committing a CL."""
+ return []
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698