Benjamin Bannier created MESOS-9112:
---------------------------------------
Summary: mesos-style reports violations on a clean checkout
Key: MESOS-9112
URL: https://issues.apache.org/jira/browse/MESOS-9112
Project: Mesos
Issue Type: Bug
Components: build, cli, python api
Reporter: Benjamin Bannier
When running {{support/mesos-style.py}} on a clean checkout of e.g.,
{{e879e920c35}} Python style violations are reported,
{noformat}
Checking 46 Python files
Using config file /home/bbannier/src/mesos/support/pylint.config
************* Module cli.plugins.base
E:119, 0: Bad option value 'R0204' (bad-option-value)
************* Module settings
E: 32, 4: No name 'VERSION' in module 'version' (no-name-in-module)
Using config file /home/bbannier/src/mesos/support/pylint.config
************* Module mesos.http
E: 25, 0: Unable to import 'urlparse' (import-error)
E: 87,30: Undefined variable 'xrange' (undefined-variable)
Using config file /home/bbannier/src/mesos/support/pylint.config
************* Module apply-reviews
R: 99, 0: Either all return statements in a function should return an
expression, or none of them should. (inconsistent-return-statements)
C:124, 9: Do not use `len(SEQUENCE)` to determine if a sequence is empty
(len-as-condition)
R:302, 4: Unnecessary "else" after "return" (no-else-return)
************* Module generate-endpoint-help
R:215, 4: Unnecessary "else" after "return" (no-else-return)
************* Module verify-reviews
C:261, 7: Do not use `len(SEQUENCE)` to determine if a sequence is empty
(len-as-condition)
Total errors found: 9
{noformat}
I would expect a clean checkout to not report any violations.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
