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

Unified Diff: presubmit_support.py

Issue 113859: Add a unit test for input_api.change.tags. (Closed)
Patch Set: Will I get it right?? Created 11 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 | « no previous file | tests/presubmit_unittest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: presubmit_support.py
diff --git a/presubmit_support.py b/presubmit_support.py
index a7b58c8a1798333206d7f2d630d4a8a612c2b545..de69727242025239c0812c7fed854f4e0ed4bf97 100755
--- a/presubmit_support.py
+++ b/presubmit_support.py
@@ -661,17 +661,17 @@ def DoPresubmitChecks(change_info,
"""
presubmit_files = ListRelevantPresubmitFiles(change_info.FileList())
if not presubmit_files and verbose:
- output_stream.write("Warning, no presubmit.py found.")
+ output_stream.write("Warning, no presubmit.py found.\n")
results = []
executer = PresubmitExecuter(change_info, committing)
if default_presubmit:
if verbose:
- output_stream.write("Running default presubmit script")
+ output_stream.write("Running default presubmit script.\n")
results += executer.ExecPresubmitScript(default_presubmit, 'PRESUBMIT.py')
for filename in presubmit_files:
filename = os.path.abspath(filename)
if verbose:
- output_stream.write("Running %s" % filename)
+ output_stream.write("Running %s\n" % filename)
# Accept CRLF presubmit script.
presubmit_script = gcl.ReadFile(filename, 'rU')
results += executer.ExecPresubmitScript(presubmit_script, filename)
« no previous file with comments | « no previous file | tests/presubmit_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698