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

Side by Side Diff: PRESUBMIT.py

Issue 739303003: Cleanup pylint errors (Closed) Base URL: http://gyp.googlecode.com/svn/trunk
Patch Set: it's clean now Created 6 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | buildbot/buildbot_run.py » ('j') | pylib/gyp/MSVSSettings.py » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 Google Inc. All rights reserved. 1 # Copyright (c) 2012 Google Inc. 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 4
5 5
6 """Top-level presubmit script for GYP. 6 """Top-level presubmit script for GYP.
7 7
8 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts 8 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
9 for more details about the presubmit API built into gcl. 9 for more details about the presubmit API built into gcl.
10 """ 10 """
(...skipping 20 matching lines...) Expand all
31 'W0612', 31 'W0612',
32 # Operator not preceded/followed by space. 32 # Operator not preceded/followed by space.
33 'C0323', 33 'C0323',
34 'C0322', 34 'C0322',
35 # Unnecessary semicolon. 35 # Unnecessary semicolon.
36 'W0301', 36 'W0301',
37 # Unused argument. 37 # Unused argument.
38 'W0613', 38 'W0613',
39 # String has no effect (docstring in wrong place). 39 # String has no effect (docstring in wrong place).
40 'W0105', 40 'W0105',
41 # map/filter on lambda could be replaced by comprehension.
42 'W0110',
43 # Use of eval.
44 'W0123',
41 # Comma not followed by space. 45 # Comma not followed by space.
42 'C0324', 46 'C0324',
43 # Access to a protected member. 47 # Access to a protected member.
44 'W0212', 48 'W0212',
45 # Bad indent. 49 # Bad indent.
46 'W0311', 50 'W0311',
47 # Line too long. 51 # Line too long.
48 'C0301', 52 'C0301',
49 # Undefined variable. 53 # Undefined variable.
50 'E0602', 54 'E0602',
51 # Not exception type specified. 55 # Not exception type specified.
52 'W0702', 56 'W0702',
53 # No member of that name. 57 # No member of that name.
54 'E1101', 58 'E1101',
55 # Dangerous default {}. 59 # Dangerous default {}.
56 'W0102', 60 'W0102',
61 # Cyclic import.
62 'R0401',
57 # Others, too many to sort. 63 # Others, too many to sort.
58 'W0201', 'W0232', 'E1103', 'W0621', 'W0108', 'W0223', 'W0231', 64 'W0201', 'W0232', 'E1103', 'W0621', 'W0108', 'W0223', 'W0231',
59 'R0201', 'E0101', 'C0321', 65 'R0201', 'E0101', 'C0321',
60 # ************* Module copy 66 # ************* Module copy
61 # W0104:427,12:_test.odict.__setitem__: Statement seems to have no effect 67 # W0104:427,12:_test.odict.__setitem__: Statement seems to have no effect
62 'W0104', 68 'W0104',
63 ] 69 ]
64 70
65 71
66 def CheckChangeOnUpload(input_api, output_api): 72 def CheckChangeOnUpload(input_api, output_api):
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
109 output_api, 115 output_api,
110 black_list=blacklist, 116 black_list=blacklist,
111 disabled_warnings=PYLINT_DISABLED_WARNINGS)) 117 disabled_warnings=PYLINT_DISABLED_WARNINGS))
112 finally: 118 finally:
113 sys.path = old_sys_path 119 sys.path = old_sys_path
114 return report 120 return report
115 121
116 122
117 def GetPreferredTrySlaves(): 123 def GetPreferredTrySlaves():
118 return ['gyp-win32', 'gyp-win64', 'gyp-linux', 'gyp-mac', 'gyp-android'] 124 return ['gyp-win32', 'gyp-win64', 'gyp-linux', 'gyp-mac', 'gyp-android']
OLDNEW
« no previous file with comments | « no previous file | buildbot/buildbot_run.py » ('j') | pylib/gyp/MSVSSettings.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698