OLD | NEW |
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
4 import os | 4 import os |
5 import sys | 5 import sys |
6 | 6 |
7 PYLINT_BLACKLIST = [] | 7 PYLINT_BLACKLIST = [] |
8 PYLINT_DISABLED_WARNINGS = ['R0923', 'R0201', 'E1101'] | 8 PYLINT_DISABLED_WARNINGS = ['R0923', 'R0201', 'E1101'] |
9 | 9 |
10 def _CommonChecks(input_api, output_api): | 10 def _CommonChecks(input_api, output_api): |
11 results = [] | 11 results = [] |
12 | 12 |
13 # TODO(nduca): This should call update_docs.IsUpdateDocsNeeded(). | 13 # TODO(nduca): This should call update_docs.IsUpdateDocsNeeded(). |
14 # Disabled due to crbug.com/255326. | 14 # Disabled due to crbug.com/255326. |
15 if False: | 15 if False: |
16 update_docs_path = os.path.join( | 16 update_docs_path = os.path.join( |
17 input_api.PresubmitLocalPath(), 'update_docs') | 17 input_api.PresubmitLocalPath(), 'update_docs') |
18 assert os.path.exists(update_docs_path) | 18 assert os.path.exists(update_docs_path) |
19 results.append(output_api.PresubmitError( | 19 results.append(output_api.PresubmitError( |
20 'Docs are stale. Please run:\n' + | 20 'Docs are stale. Please run:\n' + |
21 '$ %s' % os.path.abspath(update_docs_path))) | 21 '$ %s' % os.path.abspath(update_docs_path))) |
22 | 22 |
23 # Importing telemetry.web_components actually brings tvcm into the path. | |
24 import telemetry.web_components # pylint: disable=W0612 | |
25 from tvcm import presubmit_checker | |
26 checker = presubmit_checker.PresubmitChecker(input_api, output_api) | |
27 results += checker.RunChecks() | |
28 | |
29 results.extend(input_api.canned_checks.RunPylint( | 23 results.extend(input_api.canned_checks.RunPylint( |
30 input_api, output_api, | 24 input_api, output_api, |
31 black_list=PYLINT_BLACKLIST, | 25 black_list=PYLINT_BLACKLIST, |
32 disabled_warnings=PYLINT_DISABLED_WARNINGS)) | 26 disabled_warnings=PYLINT_DISABLED_WARNINGS)) |
33 return results | 27 return results |
34 | 28 |
35 def GetPathsToPrepend(input_api): | 29 def GetPathsToPrepend(input_api): |
36 return [input_api.PresubmitLocalPath()] | 30 return [input_api.PresubmitLocalPath()] |
37 | 31 |
38 def RunWithPrependedPath(prepended_path, fn, *args): | 32 def RunWithPrependedPath(prepended_path, fn, *args): |
(...skipping 11 matching lines...) Expand all Loading... |
50 results.extend(_CommonChecks(input_api, output_api)) | 44 results.extend(_CommonChecks(input_api, output_api)) |
51 return results | 45 return results |
52 return RunWithPrependedPath(GetPathsToPrepend(input_api), go) | 46 return RunWithPrependedPath(GetPathsToPrepend(input_api), go) |
53 | 47 |
54 def CheckChangeOnCommit(input_api, output_api): | 48 def CheckChangeOnCommit(input_api, output_api): |
55 def go(): | 49 def go(): |
56 results = [] | 50 results = [] |
57 results.extend(_CommonChecks(input_api, output_api)) | 51 results.extend(_CommonChecks(input_api, output_api)) |
58 return results | 52 return results |
59 return RunWithPrependedPath(GetPathsToPrepend(input_api), go) | 53 return RunWithPrependedPath(GetPathsToPrepend(input_api), go) |
OLD | NEW |