Chromium Code Reviews| Index: third_party/gsutil/boto/pylintrc |
| diff --git a/pylintrc b/third_party/gsutil/boto/pylintrc |
| similarity index 50% |
| copy from pylintrc |
| copy to third_party/gsutil/boto/pylintrc |
| index b76c81d71321625d5a71fb1ac889353d4af92bdf..44ed07796f5fa5e4cb89d6bff3128d29306e16c2 100644 |
| --- a/pylintrc |
| +++ b/third_party/gsutil/boto/pylintrc |
| @@ -1,22 +1,30 @@ |
| +# lint Python modules using external checkers. |
|
M-A Ruel
2013/02/27 21:40:08
Remove this file, setup, requirements, tests/, any
|
| +# |
| +# This is the main checker controlling the other ones and the reports |
| +# generation. It is itself both a raw checker and an astng checker in order |
| +# to: |
| +# * handle message activation / deactivation at the module level |
| +# * handle some basic but necessary stats'data (number of classes, methods...) |
| +# |
| [MASTER] |
| + |
| # Specify a configuration file. |
| #rcfile= |
| -# Python code to execute, usually for sys.path manipulation such as |
| -# pygtk.require(). |
| -#init-hook= |
| - |
| # Profiled execution. |
| profile=no |
| -# Add files or directories to the blacklist. They should be base names, not |
| -# paths. |
| -ignore=CVS |
| +# Add <file or directory> to the black list. It should be a base name, not a |
| +# path. You may set this option multiple times. |
| +ignore=.svn |
| # Pickle collected data for later comparisons. |
| persistent=yes |
| +# Set the cache size for astng objects. |
| +cache-size=500 |
| + |
| # List of plugins (as comma separated values of python modules names) to load, |
| # usually to register additional checkers. |
| load-plugins= |
| @@ -24,51 +32,32 @@ load-plugins= |
| [MESSAGES CONTROL] |
| -# Enable the message, report, category or checker with the given id(s). You can |
| -# either give multiple identifier separated by comma (,) or put this option |
| -# multiple time. |
| -#enable= |
| - |
| -# Disable the message, report, category or checker with the given id(s). You |
| -# can either give multiple identifier separated by comma (,) or put this option |
| -# multiple time (only on the command line, not in the configuration file where |
| -# it should appear only once). |
| -# CHANGED: |
| -# C0103: Invalid name "" |
| -# C0111: Missing docstring |
| -# C0302: Too many lines in module (N) |
| -# I0010: Unable to consider inline option '' |
| -# I0011: Locally disabling WNNNN |
| -# |
| -# R0801: Similar lines in N files |
| -# R0901: Too many ancestors (8/7) |
| -# R0902: Too many instance attributes (N/7) |
| -# R0903: Too few public methods (N/2) |
| -# R0904: Too many public methods (N/20) |
| -# R0911: Too many return statements (N/6) |
| -# R0912: Too many branches (N/12) |
| -# R0913: Too many arguments (N/5) |
| -# R0914: Too many local variables (N/15) |
| -# R0915: Too many statements (N/50) |
| -# R0921: Abstract class not referenced |
| -# R0922: Abstract class is only referenced 1 times |
| -# W0122: Use of the exec statement |
| -# W0141: Used builtin function '' |
| -# W0142: Used * or ** magic |
| -# W0402: Uses of a deprecated module 'string' |
| -# W0404: 41: Reimport 'XX' (imported line NN) |
| -# W0511: TODO |
| -# W0603: Using the global statement |
| -# W0703: Catch "Exception" |
| -# W1201: Specify string format arguments as logging function parameters |
| -disable=C0103,C0111,C0302,I0010,I0011,R0801,R0901,R0902,R0903,R0904,R0911,R0912,R0913,R0914,R0915,R0921,R0922,W0122,W0141,W0142,W0402,W0404,W0511,W0603,W0703,W1201 |
| +# Enable only checker(s) with the given id(s). This option conflict with the |
| +# disable-checker option |
| +#enable-checker= |
| + |
| +# Enable all checker(s) except those with the given id(s). This option conflict |
| +# with the disable-checker option |
| +#disable-checker= |
| + |
| +# Enable all messages in the listed categories. |
| +#enable-msg-cat= |
| + |
| +# Disable all messages in the listed categories. |
| +#disable-msg-cat= |
| +# Enable the message(s) with the given id(s). |
| +#enable-msg= |
| + |
| +# Disable the message(s) with the given id(s). |
| +# disable-msg=C0323,W0142,C0301,C0103,C0111,E0213,C0302,C0203,W0703,R0201 |
| +disable-msg=C0111,C0103,W0703,W0702 |
| [REPORTS] |
| -# Set the output format. Available formats are text, parseable, colorized, msvs |
| -# (visual studio) and html |
| -output-format=text |
| +# set the output format. Available formats are text, parseable, colorized and |
| +# html |
| +output-format=colorized |
| # Include message's id in output |
| include-ids=yes |
| @@ -78,29 +67,38 @@ include-ids=yes |
| # written in a file name "pylint_global.[txt|html]". |
| files-output=no |
| -# Tells whether to display a full report or only the messages |
| -# CHANGED: |
| -reports=no |
| +# Tells wether to display a full report or only the messages |
| +reports=yes |
| # Python expression which should return a note less than 10 (10 is the highest |
| -# note). You have access to the variables errors warning, statement which |
| -# respectively contain the number of errors / warnings messages and the total |
| +# note).You have access to the variables errors warning, statement which |
| +# respectivly contain the number of errors / warnings messages and the total |
| # number of statements analyzed. This is used by the global evaluation report |
| -# (RP0004). |
| +# (R0004). |
| evaluation=10.0 - ((float(5 * error + warning + refactor + convention) / statement) * 10) |
| # Add a comment according to your evaluation note. This is used by the global |
| -# evaluation report (RP0004). |
| +# evaluation report (R0004). |
| comment=no |
| +# Enable the report(s) with the given id(s). |
| +#enable-report= |
| + |
| +# Disable the report(s) with the given id(s). |
| +#disable-report= |
| +# checks for |
| +# * unused variables / imports |
| +# * undefined variables |
| +# * redefinition of variable from builtins or from an outer scope |
| +# * use of variable before assigment |
| +# |
| [VARIABLES] |
| -# Tells whether we should check for unused import in __init__ files. |
| -init-import=no |
| +# Tells wether we should check for unused import in __init__ files. |
| +init-import=yes |
| -# A regular expression matching the beginning of the name of dummy variables |
| -# (i.e. not used). |
| +# A regular expression matching names used for dummy variables (i.e. not used). |
| dummy-variables-rgx=_|dummy |
| # List of additional names supposed to be defined in builtins. Remember that |
| @@ -108,71 +106,47 @@ dummy-variables-rgx=_|dummy |
| additional-builtins= |
| +# try to find bugs in the code using type inference |
| +# |
| [TYPECHECK] |
| -# Tells whether missing members accessed in mixin class should be ignored. A |
| +# Tells wether missing members accessed in mixin class should be ignored. A |
| # mixin class is detected if its name ends with "mixin" (case insensitive). |
| ignore-mixin-members=yes |
| -# List of classes names for which member attributes should not be checked |
| -# (useful for classes with attributes dynamically set). |
| -ignored-classes=SQLObject,twisted.internet.reactor,hashlib,google.appengine.api.memcache |
| - |
| -# When zope mode is activated, add a predefined set of Zope acquired attributes |
| -# to generated-members. |
| +# When zope mode is activated, consider the acquired-members option to ignore |
| +# access to some undefined attributes. |
| zope=no |
| -# List of members which are set dynamically and missed by pylint inference |
| -# system, and so shouldn't trigger E0201 when accessed. Python regular |
| -# expressions are accepted. |
| -generated-members=REQUEST,acl_users,aq_parent,multiprocessing.managers.SyncManager |
| - |
| - |
| -[MISCELLANEOUS] |
| - |
| -# List of note tags to take in consideration, separated by a comma. |
| -notes=FIXME,XXX,TODO |
| - |
| - |
| -[SIMILARITIES] |
| - |
| -# Minimum lines number of a similarity. |
| -min-similarity-lines=4 |
| - |
| -# Ignore comments when computing similarities. |
| -ignore-comments=yes |
| - |
| -# Ignore docstrings when computing similarities. |
| -ignore-docstrings=yes |
| - |
| - |
| -[FORMAT] |
| - |
| -# Maximum number of characters on a single line. |
| -max-line-length=80 |
| - |
| -# Maximum number of lines in a module |
| -max-module-lines=1000 |
| - |
| -# String used as indentation unit. This is usually " " (4 spaces) or "\t" (1 |
| -# tab). |
| -# CHANGED: |
| -indent-string=' ' |
| - |
| - |
| +# List of members which are usually get through zope's acquisition mecanism and |
| +# so shouldn't trigger E0201 when accessed (need zope=yes to be considered). |
| +acquired-members=REQUEST,acl_users,aq_parent |
| + |
| + |
| +# checks for : |
| +# * doc strings |
| +# * modules / classes / functions / methods / arguments / variables name |
| +# * number of arguments, local variables, branches, returns and statements in |
| +# functions, methods |
| +# * required module attributes |
| +# * dangerous default values as arguments |
| +# * redefinition of function / method / class |
| +# * uses of the global statement |
| +# |
| [BASIC] |
| # Required attributes for module, separated by a comma |
| required-attributes= |
| -# List of builtins function names that should not be used, separated by a comma |
| -bad-functions=map,filter,apply,input |
| +# Regular expression which should only match functions or classes name which do |
| +# not require a docstring |
| +no-docstring-rgx=__.*__ |
| # Regular expression which should only match correct module names |
| module-rgx=(([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$ |
| # Regular expression which should only match correct module level names |
| -const-rgx=(([A-Z_][A-Z0-9_]*)|(__.*__))$ |
| +const-rgx=(([A-Z_][A-Z1-9_]*)|(__.*__))$ |
| # Regular expression which should only match correct class names |
| class-rgx=[A-Z_][a-zA-Z0-9]+$ |
| @@ -202,78 +176,130 @@ good-names=i,j,k,ex,Run,_ |
| # Bad variable names which should always be refused, separated by a comma |
| bad-names=foo,bar,baz,toto,tutu,tata |
| -# Regular expression which should only match functions or classes name which do |
| -# not require a docstring |
| -no-docstring-rgx=__.*__ |
| +# List of builtins function names that should not be used, separated by a comma |
| +bad-functions=apply,input |
| +# checks for sign of poor/misdesign: |
| +# * number of methods, attributes, local variables... |
| +# * size, complexity of functions, methods |
| +# |
| [DESIGN] |
| # Maximum number of arguments for function / method |
| -max-args=5 |
| - |
| -# Argument names that match this expression will be ignored. Default to name |
| -# with leading underscore |
| -ignored-argument-names=_.* |
| +max-args=12 |
| # Maximum number of locals for function / method body |
| -max-locals=15 |
| +max-locals=30 |
| # Maximum number of return / yield for function / method body |
| -max-returns=6 |
| +max-returns=12 |
| # Maximum number of branch for function / method body |
| -max-branchs=12 |
| +max-branchs=30 |
| # Maximum number of statements in function / method body |
| -max-statements=50 |
| +max-statements=60 |
| # Maximum number of parents for a class (see R0901). |
| max-parents=7 |
| # Maximum number of attributes for a class (see R0902). |
| -max-attributes=7 |
| +max-attributes=20 |
| # Minimum number of public methods for a class (see R0903). |
| -min-public-methods=2 |
| +min-public-methods=0 |
| # Maximum number of public methods for a class (see R0904). |
| max-public-methods=20 |
| +# checks for |
| +# * external modules dependencies |
| +# * relative / wildcard imports |
| +# * cyclic imports |
| +# * uses of deprecated modules |
| +# |
| +[IMPORTS] |
| + |
| +# Deprecated modules which should not be used, separated by a comma |
| +deprecated-modules=regsub,string,TERMIOS,Bastion,rexec |
| + |
| +# Create a graph of every (i.e. internal and external) dependencies in the |
| +# given file (report R0402 must not be disabled) |
| +import-graph= |
| + |
| +# Create a graph of external dependencies in the given file (report R0402 must |
| +# not be disabled) |
| +ext-import-graph= |
| + |
| +# Create a graph of internal dependencies in the given file (report R0402 must |
| +# not be disabled) |
| +int-import-graph= |
| + |
| + |
| +# checks for : |
| +# * methods without self as first argument |
| +# * overridden methods signature |
| +# * access only to existant members via self |
| +# * attributes not defined in the __init__ method |
| +# * supported interfaces implementation |
| +# * unreachable code |
| +# |
| [CLASSES] |
| # List of interface methods to ignore, separated by a comma. This is used for |
| # instance to not check methods defines in Zope's Interface base class. |
| -ignore-iface-methods=isImplementedBy,deferred,extends,names,namesAndDescriptions,queryDescriptionFor,getBases,getDescriptionFor,getDoc,getName,getTaggedValue,getTaggedValueTags,isEqualOrExtendedBy,setTaggedValue,isImplementedByInstancesOf,adaptWith,is_implemented_by |
| +# ignore-iface-methods=isImplementedBy,deferred,extends,names,namesAndDescriptions,queryDescriptionFor,getBases,getDescriptionFor,getDoc,getName,getTaggedValue,getTaggedValueTags,isEqualOrExtendedBy,setTaggedValue,isImplementedByInstancesOf,adaptWith,is_implemented_by |
| # List of method names used to declare (i.e. assign) instance attributes. |
| defining-attr-methods=__init__,__new__,setUp |
| -# List of valid names for the first argument in a class method. |
| -valid-classmethod-first-arg=cls |
| +# checks for similarities and duplicated code. This computation may be |
| +# memory / CPU intensive, so you should disable it if you experiments some |
| +# problems. |
| +# |
| +[SIMILARITIES] |
| -[IMPORTS] |
| +# Minimum lines number of a similarity. |
| +min-similarity-lines=5 |
| -# Deprecated modules which should not be used, separated by a comma |
| -deprecated-modules=regsub,string,TERMIOS,Bastion,rexec |
| +# Ignore comments when computing similarities. |
| +ignore-comments=yes |
| -# Create a graph of every (i.e. internal and external) dependencies in the |
| -# given file (report RP0402 must not be disabled) |
| -import-graph= |
| +# Ignore docstrings when computing similarities. |
| +ignore-docstrings=yes |
| -# Create a graph of external dependencies in the given file (report RP0402 must |
| -# not be disabled) |
| -ext-import-graph= |
| -# Create a graph of internal dependencies in the given file (report RP0402 must |
| -# not be disabled) |
| -int-import-graph= |
| +# checks for: |
| +# * warning notes in the code like FIXME, XXX |
| +# * PEP 263: source code with non ascii character but no encoding declaration |
| +# |
| +[MISCELLANEOUS] |
| + |
| +# List of note tags to take in consideration, separated by a comma. |
| +notes=FIXME,XXX,TODO,BUG: |
| -[EXCEPTIONS] |
| +# checks for : |
| +# * unauthorized constructions |
| +# * strict indentation |
| +# * line length |
| +# * use of <> instead of != |
| +# |
| +[FORMAT] |
| -# Exceptions that will emit a warning when being caught. Defaults to |
| -# "Exception" |
| -overgeneral-exceptions=Exception |
| +# Maximum number of characters on a single line. |
| +max-line-length=80 |
| + |
| +# Maximum number of lines in a module |
| +max-module-lines=1000 |
| + |
| +# String used as indentation unit. This is usually " " (4 spaces) or "\t" (1 |
| +# tab). |
| +indent-string=' ' |
| + |
| + |
| +[MESSAGES CONTROL] |
| +disable-msg=C0301,C0111,C0103,R0201,W0702,C0324 |