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

Side by Side Diff: third_party/WebKit/Tools/Scripts/webkitpy/pylintrc

Issue 2120083002: Enable stricter pylint checks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move comment so that it doesn't interfere with disable. Created 4 years, 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
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 # 2 #
3 # Redistribution and use in source and binary forms, with or without 3 # Redistribution and use in source and binary forms, with or without
4 # modification, are permitted provided that the following conditions are 4 # modification, are permitted provided that the following conditions are
5 # met: 5 # met:
6 # 6 #
7 # * Redistributions of source code must retain the above copyright 7 # * Redistributions of source code must retain the above copyright
8 # notice, this list of conditions and the following disclaimer. 8 # notice, this list of conditions and the following disclaimer.
9 # * Redistributions in binary form must reproduce the above 9 # * Redistributions in binary form must reproduce the above
10 # copyright notice, this list of conditions and the following disclaimer 10 # copyright notice, this list of conditions and the following disclaimer
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 # Enable the message, report, category or checker with the given id(s). You can 57 # Enable the message, report, category or checker with the given id(s). You can
58 # either give multiple identifier separated by comma (,) or put this option 58 # either give multiple identifier separated by comma (,) or put this option
59 # multiple time. 59 # multiple time.
60 #enable= 60 #enable=
61 61
62 # Disable the message, report, category or checker with the given id(s). You 62 # Disable the message, report, category or checker with the given id(s). You
63 # can either give multiple identifier separated by comma (,) or put this option 63 # can either give multiple identifier separated by comma (,) or put this option
64 # multiple time (only on the command line, not in the configuration file where 64 # multiple time (only on the command line, not in the configuration file where
65 # it should appear only once). 65 # it should appear only once).
66 # CHANGED: 66 # CHANGED:
67 # The "design" disable below disables all of the "too-many-*" checks,
68 # e.g. too-many-arguments, too-many-branches.
67 disable= 69 disable=
68 invalid-name, 70 design,
69 missing-docstring, 71 missing-docstring,
70 line-too-long,
71 too-many-lines,
72 bad-inline-option,
73 locally-disabled, 72 locally-disabled,
74 no-self-use,
75 duplicate-code,
76 too-many-ancestors,
77 too-many-instance-attributes,
78 too-few-public-methods,
79 too-many-public-methods,
80 too-many-return-statements,
81 too-many-branches,
82 too-many-arguments,
83 too-many-locals,
84 too-many-statements,
85 abstract-class-not-used,
86 abstract-class-little-used,
87 exec-used,
88 bad-builtin,
89 star-args, 73 star-args,
90 protected-access,
91 wildcard-import, 74 wildcard-import,
92 deprecated-module,
93 reimported,
94 fixme, 75 fixme,
95 global-statement, 76 global-statement,
96 unused-wildcard-import,
97 broad-except,
98 logging-not-lazy,
99 77
100 78
101 [REPORTS] 79 [REPORTS]
102 80
103 # Set the output format. Available formats are text, parseable, colorized, msvs 81 # Set the output format. Available formats are text, parseable, colorized, msvs
104 # (visual studio) and html 82 # (visual studio) and html
105 output-format=text 83 output-format=text
106 84
107 # Put messages in a separate file for each module / package specified on the 85 # Put messages in a separate file for each module / package specified on the
108 # command line instead of printing them on stdout. Reports (if any) will be 86 # command line instead of printing them on stdout. Reports (if any) will be
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
173 # Ignore comments when computing similarities. 151 # Ignore comments when computing similarities.
174 ignore-comments=yes 152 ignore-comments=yes
175 153
176 # Ignore docstrings when computing similarities. 154 # Ignore docstrings when computing similarities.
177 ignore-docstrings=yes 155 ignore-docstrings=yes
178 156
179 157
180 [FORMAT] 158 [FORMAT]
181 159
182 # Maximum number of characters on a single line. 160 # Maximum number of characters on a single line.
183 max-line-length=200 161 max-line-length=132
184 162
185 # Maximum number of lines in a module 163 # Maximum number of lines in a module
186 max-module-lines=1000 164 max-module-lines=4000
187 165
188 # String used as indentation unit. This is usually " " (4 spaces) or "\t" (1 166 # String used as indentation unit. This is usually " " (4 spaces) or "\t" (1
189 # tab). 167 # tab).
190 # CHANGED: 168 # CHANGED:
191 indent-string=' ' 169 indent-string=' '
192 170
193 171
194 [BASIC] 172 [BASIC]
195 173
196 # Required attributes for module, separated by a comma 174 # Required attributes for module, separated by a comma
197 required-attributes= 175 required-attributes=
198 176
199 # List of builtins function names that should not be used, separated by a comma 177 # List of builtins function names that should not be used, separated by a comma
200 bad-functions=map,filter,apply,input 178 bad-functions=apply
201 179
202 # Regular expression which should only match correct module names 180 # Regular expression which should only match correct module names
203 module-rgx=(([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$ 181 module-rgx=(([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+)|([a-z-][a-z0-9-]*))$
204 182
205 # Regular expression which should only match correct module level names 183 # Regular expression which should only match correct module level names
206 const-rgx=(([A-Z_][A-Z0-9_]*)|(__.*__))$ 184 const-rgx=(([A-Z_][A-Z0-9_]*)|(__.*__))$
207 185
208 # Regular expression which should only match correct class names 186 # Regular expression which should only match correct class names
209 class-rgx=[A-Z_][a-zA-Z0-9]+$ 187 class-rgx=[A-Z_][a-zA-Z0-9]+$
210 188
211 # Regular expression which should only match correct function names 189 # Regular expression which should only match correct function names
212 function-rgx=[a-z_][a-z0-9_]{2,30}$ 190 function-rgx=[a-z_][a-z0-9_]{2,30}$
213 191
214 # Regular expression which should only match correct method names 192 # Regular expression which should only match correct method names
215 method-rgx=[a-z_][a-z0-9_]{2,30}$ 193 method-rgx=[a-z_][a-z0-9_]{2,30}$
216 194
217 # Regular expression which should only match correct instance attribute names 195 # Regular expression which should only match correct instance attribute names
218 attr-rgx=[a-z_][a-z0-9_]{2,30}$ 196 attr-rgx=[a-z_][a-z0-9_]{1,30}$
219 197
220 # Regular expression which should only match correct argument names 198 # Regular expression which should only match correct argument names
221 argument-rgx=[a-z_][a-z0-9_]{2,30}$ 199 argument-rgx=[a-z_][a-z0-9_]{1,30}$
222 200
223 # Regular expression which should only match correct variable names 201 # Regular expression which should only match correct variable names
224 variable-rgx=[a-z_][a-z0-9_]{2,30}$ 202 variable-rgx=[a-z_][a-z0-9_]{1,30}$
225 203
226 # Regular expression which should only match correct list comprehension / 204 # Regular expression which should only match correct list comprehension /
227 # generator expression variable names 205 # generator expression variable names
228 inlinevar-rgx=[A-Za-z_][A-Za-z0-9_]*$ 206 inlinevar-rgx=[A-Za-z_][A-Za-z0-9_]*$
229 207
230 # Good variable names which should always be accepted, separated by a comma 208 # Good variable names which should always be accepted, separated by a comma
231 good-names=i,j,k,ex,Run,_ 209 good-names=i,j,k,ex,Run,_,_log
232 210
233 # Bad variable names which should always be refused, separated by a comma 211 # Bad variable names which should always be refused, separated by a comma
234 bad-names=foo,bar,baz,toto,tutu,tata 212 bad-names=foo,bar,baz,toto,tutu,tata
235 213
236 # Regular expression which should only match functions or classes name which do 214 # Regular expression which should only match functions or classes name which do
237 # not require a docstring 215 # not require a docstring
238 no-docstring-rgx=__.*__ 216 no-docstring-rgx=__.*__
239 217
240 218
241 [DESIGN] 219 [DESIGN]
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
301 # Create a graph of internal dependencies in the given file (report RP0402 must 279 # Create a graph of internal dependencies in the given file (report RP0402 must
302 # not be disabled) 280 # not be disabled)
303 int-import-graph= 281 int-import-graph=
304 282
305 283
306 [EXCEPTIONS] 284 [EXCEPTIONS]
307 285
308 # Exceptions that will emit a warning when being caught. Defaults to 286 # Exceptions that will emit a warning when being caught. Defaults to
309 # "Exception" 287 # "Exception"
310 overgeneral-exceptions=Exception 288 overgeneral-exceptions=Exception
OLDNEW
« 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