|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH 1/7] tools/python: Drop pylintrc
On Tue, Mar 14, 2023 at 02:15:14PM +0000, Andrew Cooper wrote: > This was added in 2004 in c/s b7d4a69f0ccb5 and has never been referenced > since. Given the the commit message of simply "Added .", it was quite > possibly a mistake in the first place. > > Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Ideally, we would have a pylint integrated into CI, but given frequency of changes to that code, I don't think its worth it, so: Acked-by: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx> > --- > CC: Wei Liu <wl@xxxxxxx> > CC: Anthony PERARD <anthony.perard@xxxxxxxxxx> > CC: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx> > CC: Bernhard Kaindl <bernhard.kaindl@xxxxxxxxxx> > --- > tools/python/pylintrc | 307 ------------------------------------------ > 1 file changed, 307 deletions(-) > delete mode 100644 tools/python/pylintrc > > diff --git a/tools/python/pylintrc b/tools/python/pylintrc > deleted file mode 100644 > index efc4b0b3b2dd..000000000000 > --- a/tools/python/pylintrc > +++ /dev/null > @@ -1,307 +0,0 @@ > -# lint Python modules using external checkers. > -# > > -# This is the main checker controling 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...) > -# > > -# This checker also defines the following reports: > > -# * R0001: Total errors / warnings > > -# * R0002: % errors / warnings by module > > -# * R0003: Messages > > -# * R0004: Global evaluation > > -# > -[MASTER] > -# 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=CVS > - > -# Pickle collected data for later comparisons. > -persistent=yes > - > -# Set the cache size for astng objects. > -cache-size=500 > - > - > - > -[REPORTS] > -# Tells wether to display a full report or only the messages > -reports=yes > - > -# Use HTML as output format instead of text > -html=no > - > -# Use a parseable text output format, so your favorite text editor will be > able > -# to jump to the line corresponding to a message. > -parseable=no > - > -# Colorizes text output using ansi escape codes > -color=no > - > -# Put messages in a separate file for each module / package specified on the > -# command line instead of printing them on stdout. Reports (if any) will be > -# written in a file name "pylint_global.[txt|html]". > -files-output=no > - > -# 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 > -# respectivly contain the number of errors / warnings messages and the total > -# number of statements analyzed. This is used by the global evaluation report > -# (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 (R0004). > -comment=no > - > -# Include message's id in output > -include-ids=yes > - > - > - > -# checks for > -# * unused variables / imports > > -# * undefined variables > > -# * redefinition of variable from builtins or from an outer scope > > -# * use of variable before assigment > > -# > -[VARIABLES] > -# Enable / disable this checker > -enable-variables=yes > - > -# Tells wether we should check for unused import in __init__ files. > -init-import=no > - > -# List of variable names used for dummy variables (i.e. not used). > -dummy-variables=_,_1,_2,_3,_4,_5,dummy > - > - > - > -# checks for : > -# * doc strings > > -# * modules / classes / functions / methods / arguments / variables name > > -# * number of arguments, local variables, branchs, returns and > statements in > -# functions, methods > -# * required module attributes > > -# * dangerous default values as arguments > > -# * redefinition of function / method / class > > -# * uses of the global statement > > -# > > -# This checker also defines the following reports: > > -# * R0101: Statistics by type > > -# > -[BASIC] > -# Enable / disable this checker > -enable-basic=yes > - > -# Required attributes for module, separated by a comma > -required-attributes= > - > -# Regular expression which should only match functions or classes name which > do > -# not require a docstring > -no-docstring-rgx=.* > - > -# Minimal length for module / class / function / method / argument / variable > -# names > -min-name-length=1 > - > -# 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 class names > -class-rgx=[A-Z_][a-zA-Z0-9]+$ > - > -# Regular expression which should only match correct function names > -function-rgx=[a-z_][A-Za-z0-9_]*$ > - > -# Regular expression which should only match correct method names > -method-rgx=[a-z_][A-Za-z0-9_]*$ > - > -# Regular expression which should only match correct argument names > -argument-rgx=[a-z_][A-Za-z0-9_]*$ > - > -# Regular expression which should only match correct variable names > -variable-rgx=[a-z_][A-Za-z0-9_]*$ > - > -# Good variable names which should always be accepted, separated by a comma > -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 > - > -# 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] > -# Enable / disable this checker > -enable-design=yes > - > -# Maximum number of arguments for function / method > -max-args=15 > - > -# Maximum number of locals for function / method body > -max-locals=15 > - > -# Maximum number of return / yield for function / method body > -max-returns=6 > - > -# Maximum number of branch for function / method body > -max-branchs=12 > - > -# Maximum number of statements in function / method body > -max-statements=50 > - > -# Maximum number of parents for a class (see R0901). > -max-parents=7 > - > -# Maximum number of attributes for a class (see R0902). > -max-attributes=7 > - > -# Minimum number of public methods for a class (see R0903). > -min-public-methods=2 > - > -# Maximum number of public methods for a class (see R0904). > -max-public-methods=20 > - > - > - > -# checks for : > -# * methods without self as first argument > > -# * overriden methods signature > > -# * access only to existant members via self > > -# * attributes not defined in the __init__ method > > -# * supported interfaces implementation > > -# * unreachable code > > -# > -[CLASSES] > -# Enable / disable this checker > -enable-classes=yes > - > -# 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 > - > -# 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 > - > - > - > -# checks for > -# * external modules dependencies > > -# * relative / wildcard imports > > -# * cyclic imports > > -# * uses of deprecated modules > -# > > -# This checker also defines the following reports: > > -# * R0401: External dependencies > > -# * R0402: Modules dependencies graph > > -# > -[IMPORTS] > -# Enable / disable this checker > -enable-imports=no > - > -# 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 > -# * excepts without exception filter > > -# * string exceptions > > -# > -[EXCEPTIONS] > -# Enable / disable this checker > -enable-exceptions=yes > - > - > - > -# checks for : > -# * unauthorized constructions > > -# * strict indentation > > -# * line length > > -# * use of <> instead of != > -# > -[FORMAT] > -# Enable / disable this checker > -enable-format=no > - > -# 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=' ' > - > - > - > -# does not check anything but gives some raw metrics : > -# * total number of lines > > -# * total number of code lines > > -# * total number of docstring lines > > -# * total number of comments lines > > -# * total number of empty lines > > -# > > -# This checker also defines the following reports: > > -# * R0701: Raw metrics > > -# > -[METRICS] > -# Enable / disable this checker > -enable-metrics=yes > - > - > - > -# checks for: > -# * warning notes in the code like FIXME, XXX > > -# * PEP 263: source code with non ascii character but no encoding > declaration > -# > -[MISCELLANEOUS] > -# Enable / disable this checker > -enable-miscellaneous=yes > - > -# List of note tags to take in consideration, separated by a comma. Default > to > -# FIXME, XXX, TODO > -notes=FIXME,XXX,TODO > - > - > - > -# checks for similarities and duplicated code. This computation may be > -# memory / CPU intensive, so you should disable it if you experiments > some > -# problems. > -# > > -# This checker also defines the following reports: > > -# * R0801: Duplication > > -# > -[SIMILARITIES] > -# Enable / disable this checker > -enable-similarities=yes > - > -# Minimum lines number of a similarity. > -min-similarity-lines=4 > - > -# Ignore comments when computing similarities. > -ignore-comments=yes > - > - > - > -- > 2.30.2 > -- Best Regards, Marek Marczykowski-Górecki Invisible Things Lab Attachment:
signature.asc
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |