diff --git a/presubmit_canned_checks.py b/presubmit_canned_checks.py index 68b981908..aecdd83fa 100644 --- a/presubmit_canned_checks.py +++ b/presubmit_canned_checks.py @@ -228,7 +228,7 @@ def CheckChangeHasNoCrAndHasOnlyOneEol(input_api, output_api, def _ReportErrorFileAndLine(filename, line_num, dummy_line): """Default error formatter for _FindNewViolationsOfRule.""" - return '%s, line %s' % (filename, line_num) + return '%s:%s' % (filename, line_num) def _FindNewViolationsOfRule(callable_rule, input_api, source_file_filter=None, diff --git a/tests/presubmit_unittest.py b/tests/presubmit_unittest.py index f97121a7b..34f0b489e 100755 --- a/tests/presubmit_unittest.py +++ b/tests/presubmit_unittest.py @@ -2053,7 +2053,7 @@ class CannedChecksUnittest(PresubmitTestsBase): self.assertEquals(results1[0].__class__, presubmit.OutputApi.PresubmitPromptWarning) self.assertEquals(results1[0]._long_text, - 'makefile.foo, line 46') + 'makefile.foo:46') def testCannedCheckLongLines(self): check = lambda x, y, z: presubmit_canned_checks.CheckLongLines(x, y, 10, z)