[
https://issues.apache.org/jira/browse/MESOS-9112?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Gavin updated MESOS-9112:
-------------------------
Comment: was deleted
(was: www.rtat.net)
> 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
> Assignee: Armand Grillet
> Priority: Major
> Labels: lint, python, style
> Fix For: 1.8.0
>
>
> 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)
