Home

Welcome to the maintainer-quality-tools wiki! OCA Repo health board

Travis style rules

Config files can be found in https://github.com/OCA/maintainer-quality-tools/tree/master/travis/cfg

Flake8 Rules:

• Default flake8 rules
• max-line-length = 79
• ignore unused imports (F401) in __init__.py only

Pylint Rules:

(all these can be seen with pylint2 --help-msg <msg-id>[,<msg-id>])

• E0101 return-in-init: Used when the special class method __init__ has an explicit return value.
• E1124 redundant-keyword-arg: Used when a function call would result in assigning multiple values to a function parameter, one value from a positional argument and one from a keyword argument.
• E1306 too-few-format-args: Used when a format string that uses unnamed conversion specifiers is given too few arguments
• W0101 unreachable: Used when there is some code behind a "return" or "raise" statement, which will never be accessed.
• W0102 dangerous-default-value: Used when a mutable value as list or dictionary is detected in a default value for an argument.
• W0104 pointless-statement: Used when a statement doesn't have (or at least seems to) any effect.
• W0105 pointless-string-statement: Used when a string is used as a statement (which of course has no effect).
• W0109 duplicate-key: Used when a dictionary expression binds the same key multiple times.
• W0403 relative-import: Used when an import relative to the package directory is detected.
• W0404 reimported: Used when a module is reimported multiple times.
• W0621 redefined-outer-name: Used when a variable's name hide a name defined in the outer scope.
• W0622 redefined-builtin: Used when a variable or function override a built-in.
• W1111 assignment-from-none: Used when an assignment is done on a function call but the inferred function returns nothing but None.
• I0013 file-ignored: Used to inform that the file will not be checked

Odoolint Rule:

• EO001 print statement used: Used when there is a call to print