You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Armand Grillet <ag...@mesosphere.io> on 2018/09/03 15:33:09 UTC
Review Request 68608: Fixed all Python linting issues.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68608/
-----------------------------------------------------------
Review request for mesos, Benjamin Bannier, Eric Chung, and Kevin Klues.
Bugs: MESOS-9112
https://issues.apache.org/jira/browse/MESOS-9112
Repository: mesos
Description
-------
Fixed all Python linting issues.
Diffs
-----
src/python/cli_new/bin/main.py b6f9faba741d7229a02cc5052e3b96c193df8991
src/python/cli_new/bin/settings.py f501fefa2eb29ed5d7b55828dd18cb396e313583
src/python/cli_new/lib/cli/config.py 92ff3eb96826651505e0c3e786f6292cef600bc7
src/python/cli_new/lib/cli/docopt.py 4dedd7734bf5880c780be6ffe71430ac1d57aad3
src/python/cli_new/lib/cli/plugins/__init__.py e743772afce3aad753be3cd2a923a4331bafc31c
src/python/cli_new/lib/cli/plugins/agent/main.py 4ebf8cd63a52262615dbcaf1dd79ae2951778c84
src/python/cli_new/lib/cli/plugins/base.py e01a7b2bc4d4cbabe706c8926913f43d2b4cf69c
src/python/cli_new/lib/cli/plugins/task/main.py f9d6826560bcae724c8e47794a73b69b40b955ab
src/python/cli_new/lib/cli/tests/__init__.py b7c825fc17aa6fc732b5933260dd17291126665a
src/python/cli_new/lib/cli/tests/agent.py 6e5e56554b2a556410daeffaa6fbf219adee17a2
src/python/cli_new/lib/cli/tests/base.py f58d616b59232786d3d9fb44d8dd91c5c3017568
src/python/cli_new/lib/cli/tests/task.py 3a57094b1ad340dcd630c0e59783686432d8a6a5
src/python/cli_new/lib/cli/util.py b7404318afd399b2cd54b33d6883b2e7c8734f85
src/python/lib/mesos/http.py 8a5a3680b99aec5bebcb0f116e65ae820a72328f
src/python/lib/tests/test_http.py c1b21dfcac0292c0646f16f1547ce949bea711a7
support/apply-reviews.py 4e292a91f9eb3c6d4598ba258d6aae9dcfa86fc7
support/generate-endpoint-help.py 37c4b58462ca42a42276ed1129190fc3359c01cf
support/python3/apply-reviews.py 6e1d6d784ea1508c63ac7d5bb0f168a031e0db7c
support/python3/common.py ed62c675fa20fb638b9e3aa63c9a55cae39fe9d7
support/python3/generate-endpoint-help.py 6214d2efa41ca131ff47a52fbcef90fa245e1ed5
support/python3/mesos-style.py 845ec73670961057d2d2d6ecceec99b0f087f2fa
support/python3/push-commits.py 7d883505a535e9e3745dbc6dba795bcb6be9d938
support/python3/test-upgrade.py a1745bd146d369da83f2a47c69d7f4d2c142dfa8
support/verify-reviews.py c86db35c3231af8f1493519759a02cbaf7ef0209
Diff: https://reviews.apache.org/r/68608/diff/1/
Testing
-------
Before, `./support/mesos-style.py`:
```
Checking 12 Python files
Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
************* Module verify-reviews
C:261, 7: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
************* Module generate-endpoint-help
R:215, 4: Unnecessary "else" after "return" (no-else-return)
************* 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)
Total errors found: 5
```
After:
```
Checking 12 Python files
Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
Total errors found: 0
```
Before, `python3 ./support/python3/mesos-style.py`:
```
Checking 47 Python files
************* Module cli.tests.agent
lib/cli/tests/agent.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
************* Module cli.plugins.task.main
lib/cli/plugins/task/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
************* Module cli.util
lib/cli/util.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
lib/cli/util.py:282:0: R0205: Class 'Table' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
************* Module cli.plugins.base
lib/cli/plugins/base.py:119:0: E0012: Bad option value 'R0204' (bad-option-value)
lib/cli/plugins/base.py:69:0: R0205: Class 'PluginBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
lib/cli/plugins/base.py:115:11: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
************* Module cli.tests.base
lib/cli/tests/base.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
lib/cli/tests/base.py:69:8: R1705: Unnecessary "else" after "return" (no-else-return)
lib/cli/tests/base.py:81:0: R0205: Class 'Executable' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
************* Module settings
bin/settings.py:32:4: E0611: No name 'VERSION' in module 'version' (no-name-in-module)
************* Module cli.plugins.agent.main
lib/cli/plugins/agent/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
************* Module cli.config
lib/cli/config.py:31:0: R0205: Class 'Config' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
************* Module cli.tests.task
lib/cli/tests/task.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
************* Module main
bin/main.py:70:7: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
************* Module tests.test_http
tests/test_http.py:265:8: R1705: Unnecessary "else" after "return" (no-else-return)
tests/test_http.py:271:8: R1705: Unnecessary "else" after "return" (no-else-return)
************* Module mesos.http
mesos/http.py:79:0: R0205: Class 'Resource' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
************* Module test-upgrade
support/python3/test-upgrade.py:32:0: R0205: Class 'Process' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
support/python3/test-upgrade.py:65:15: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
************* Module apply-reviews
support/python3/apply-reviews.py:83:4: R1705: Unnecessary "elif" after "return" (no-else-return)
support/python3/apply-reviews.py:133:8: R1705: Unnecessary "else" after "return" (no-else-return)
support/python3/apply-reviews.py:304:4: R1705: Unnecessary "elif" after "return" (no-else-return)
************* Module common
support/python3/common.py:38:0: R0205: Class 'ReviewBoardHandler' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
************* Module generate-endpoint-help
support/python3/generate-endpoint-help.py:84:0: R0205: Class 'Subprocess' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
************* Module mesos-style
support/python3/mesos-style.py:528:0: C0301: Line too long (84/80) (line-too-long)
support/python3/mesos-style.py:551:0: C0301: Line too long (90/80) (line-too-long)
support/python3/mesos-style.py:557:0: C0301: Line too long (86/80) (line-too-long)
support/python3/mesos-style.py:566:0: C0301: Line too long (104/80) (line-too-long)
support/python3/mesos-style.py:155:0: W1401: Anomalous backslash in string: '\S'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
support/python3/mesos-style.py:29:0: R0205: Class 'LinterBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
support/python3/mesos-style.py:218:12: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
support/python3/mesos-style.py:522:4: W0621: Redefining name 'build_virtualenv' from outer scope (line 512) (redefined-outer-name)
************* Module push-commits
support/python3/push-commits.py:47:0: W0311: Bad indentation. Found 2 spaces, expected 4 (bad-indentation)
support/python3/push-commits.py:54:0: C0330: Wrong continued indentation (add 1 space).
'rev-list',
^| (bad-continuation)
support/python3/push-commits.py:55:0: C0330: Wrong continued indentation (add 1 space).
'--reverse',
^| (bad-continuation)
support/python3/push-commits.py:56:0: C0330: Wrong continued indentation (add 1 space).
revision_range]).strip().split('\n')
^| (bad-continuation)
support/python3/push-commits.py:59:0: C0330: Wrong continued indentation (add 1 space).
'--no-pager',
^| (bad-continuation)
support/python3/push-commits.py:60:0: C0330: Wrong continued indentation (add 1 space).
'show',
^| (bad-continuation)
support/python3/push-commits.py:61:0: C0330: Wrong continued indentation (add 1 space).
'--no-color',
^| (bad-continuation)
support/python3/push-commits.py:62:0: C0330: Wrong continued indentation (add 1 space).
'--no-patch',
^| (bad-continuation)
support/python3/push-commits.py:63:0: C0330: Wrong continued indentation (add 1 space).
rev]).strip()
^| (bad-continuation)
support/python3/push-commits.py:92:0: C0330: Wrong continued indentation (add 1 space).
'close',
^| (bad-continuation)
support/python3/push-commits.py:93:0: C0330: Wrong continued indentation (add 1 space).
'--description',
^| (bad-continuation)
support/python3/push-commits.py:94:0: C0330: Wrong continued indentation (add 1 space).
commit_log,
^| (bad-continuation)
support/python3/push-commits.py:95:0: C0330: Wrong continued indentation (add 1 space).
review_id])
^| (bad-continuation)
support/python3/push-commits.py:127:0: C0330: Wrong continued indentation (add 1 space).
'rev-parse',
^| (bad-continuation)
support/python3/push-commits.py:128:0: C0330: Wrong continued indentation (add 1 space).
'--abbrev-ref',
^| (bad-continuation)
support/python3/push-commits.py:129:0: C0330: Wrong continued indentation (add 1 space).
'master@{upstream}']).strip()
^| (bad-continuation)
support/python3/push-commits.py:145:0: C0330: Wrong continued indentation (add 1 space).
'config',
^| (bad-continuation)
support/python3/push-commits.py:146:0: C0330: Wrong continued indentation (add 1 space).
'--get',
^| (bad-continuation)
support/python3/push-commits.py:147:0: C0330: Wrong continued indentation (add 1 space).
'branch.master.remote']).strip()
^| (bad-continuation)
support/python3/push-commits.py:153:0: C0330: Wrong continued indentation (add 1 space).
'push',
^| (bad-continuation)
support/python3/push-commits.py:154:0: C0330: Wrong continued indentation (add 1 space).
'--dry-run',
^| (bad-continuation)
support/python3/push-commits.py:155:0: C0330: Wrong continued indentation (add 1 space).
remote,
^| (bad-continuation)
support/python3/push-commits.py:156:0: C0330: Wrong continued indentation (add 1 space).
'master:master'])
^| (bad-continuation)
support/python3/push-commits.py:159:0: C0330: Wrong continued indentation (add 1 space).
'push',
^| (bad-continuation)
support/python3/push-commits.py:160:0: C0330: Wrong continued indentation (add 1 space).
remote,
^| (bad-continuation)
support/python3/push-commits.py:161:0: C0330: Wrong continued indentation (add 1 space).
'master:master'])
^| (bad-continuation)
```
After:
```
Checking 47 Python files
Total errors found: 0
```
Thanks,
Armand Grillet
Re: Review Request 68608: Fixed all Python linting issues.
Posted by Armand Grillet <ag...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68608/#review208264
-----------------------------------------------------------
support/python3/mesos-style.py
Line 170 (original), 173 (patched)
<https://reviews.apache.org/r/68608/#comment291999>
Please update the docs.
support/python3/mesos-style.py
Line 196 (original), 199 (patched)
<https://reviews.apache.org/r/68608/#comment292000>
`i` is not `int`.
- Armand Grillet
On Sept. 3, 2018, 5:33 p.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68608/
> -----------------------------------------------------------
>
> (Updated Sept. 3, 2018, 5:33 p.m.)
>
>
> Review request for mesos, Benjamin Bannier, Eric Chung, and Kevin Klues.
>
>
> Bugs: MESOS-9112
> https://issues.apache.org/jira/browse/MESOS-9112
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Fixed all Python linting issues.
>
>
> Diffs
> -----
>
> src/python/cli_new/bin/main.py b6f9faba741d7229a02cc5052e3b96c193df8991
> src/python/cli_new/bin/settings.py f501fefa2eb29ed5d7b55828dd18cb396e313583
> src/python/cli_new/lib/cli/config.py 92ff3eb96826651505e0c3e786f6292cef600bc7
> src/python/cli_new/lib/cli/docopt.py 4dedd7734bf5880c780be6ffe71430ac1d57aad3
> src/python/cli_new/lib/cli/plugins/__init__.py e743772afce3aad753be3cd2a923a4331bafc31c
> src/python/cli_new/lib/cli/plugins/agent/main.py 4ebf8cd63a52262615dbcaf1dd79ae2951778c84
> src/python/cli_new/lib/cli/plugins/base.py e01a7b2bc4d4cbabe706c8926913f43d2b4cf69c
> src/python/cli_new/lib/cli/plugins/task/main.py f9d6826560bcae724c8e47794a73b69b40b955ab
> src/python/cli_new/lib/cli/tests/__init__.py b7c825fc17aa6fc732b5933260dd17291126665a
> src/python/cli_new/lib/cli/tests/agent.py 6e5e56554b2a556410daeffaa6fbf219adee17a2
> src/python/cli_new/lib/cli/tests/base.py f58d616b59232786d3d9fb44d8dd91c5c3017568
> src/python/cli_new/lib/cli/tests/task.py 3a57094b1ad340dcd630c0e59783686432d8a6a5
> src/python/cli_new/lib/cli/util.py b7404318afd399b2cd54b33d6883b2e7c8734f85
> src/python/lib/mesos/http.py 8a5a3680b99aec5bebcb0f116e65ae820a72328f
> src/python/lib/tests/test_http.py c1b21dfcac0292c0646f16f1547ce949bea711a7
> support/apply-reviews.py 4e292a91f9eb3c6d4598ba258d6aae9dcfa86fc7
> support/generate-endpoint-help.py 37c4b58462ca42a42276ed1129190fc3359c01cf
> support/python3/apply-reviews.py 6e1d6d784ea1508c63ac7d5bb0f168a031e0db7c
> support/python3/common.py ed62c675fa20fb638b9e3aa63c9a55cae39fe9d7
> support/python3/generate-endpoint-help.py 6214d2efa41ca131ff47a52fbcef90fa245e1ed5
> support/python3/mesos-style.py 845ec73670961057d2d2d6ecceec99b0f087f2fa
> support/python3/push-commits.py 7d883505a535e9e3745dbc6dba795bcb6be9d938
> support/python3/test-upgrade.py a1745bd146d369da83f2a47c69d7f4d2c142dfa8
> support/verify-reviews.py c86db35c3231af8f1493519759a02cbaf7ef0209
>
>
> Diff: https://reviews.apache.org/r/68608/diff/1/
>
>
> Testing
> -------
>
> Before, `./support/mesos-style.py`:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> ************* Module verify-reviews
> C:261, 7: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
> ************* Module generate-endpoint-help
> R:215, 4: Unnecessary "else" after "return" (no-else-return)
> ************* 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)
> Total errors found: 5
> ```
> After:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> Total errors found: 0
> ```
>
> Before, `python3 ./support/python3/mesos-style.py`:
> ```
> Checking 47 Python files
> ************* Module cli.tests.agent
> lib/cli/tests/agent.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.plugins.task.main
> lib/cli/plugins/task/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.util
> lib/cli/util.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/util.py:282:0: R0205: Class 'Table' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.plugins.base
> lib/cli/plugins/base.py:119:0: E0012: Bad option value 'R0204' (bad-option-value)
> lib/cli/plugins/base.py:69:0: R0205: Class 'PluginBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> lib/cli/plugins/base.py:115:11: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module cli.tests.base
> lib/cli/tests/base.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/tests/base.py:69:8: R1705: Unnecessary "else" after "return" (no-else-return)
> lib/cli/tests/base.py:81:0: R0205: Class 'Executable' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module settings
> bin/settings.py:32:4: E0611: No name 'VERSION' in module 'version' (no-name-in-module)
> ************* Module cli.plugins.agent.main
> lib/cli/plugins/agent/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.config
> lib/cli/config.py:31:0: R0205: Class 'Config' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.tests.task
> lib/cli/tests/task.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module main
> bin/main.py:70:7: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module tests.test_http
> tests/test_http.py:265:8: R1705: Unnecessary "else" after "return" (no-else-return)
> tests/test_http.py:271:8: R1705: Unnecessary "else" after "return" (no-else-return)
> ************* Module mesos.http
> mesos/http.py:79:0: R0205: Class 'Resource' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module test-upgrade
> support/python3/test-upgrade.py:32:0: R0205: Class 'Process' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/test-upgrade.py:65:15: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module apply-reviews
> support/python3/apply-reviews.py:83:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> support/python3/apply-reviews.py:133:8: R1705: Unnecessary "else" after "return" (no-else-return)
> support/python3/apply-reviews.py:304:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> ************* Module common
> support/python3/common.py:38:0: R0205: Class 'ReviewBoardHandler' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module generate-endpoint-help
> support/python3/generate-endpoint-help.py:84:0: R0205: Class 'Subprocess' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module mesos-style
> support/python3/mesos-style.py:528:0: C0301: Line too long (84/80) (line-too-long)
> support/python3/mesos-style.py:551:0: C0301: Line too long (90/80) (line-too-long)
> support/python3/mesos-style.py:557:0: C0301: Line too long (86/80) (line-too-long)
> support/python3/mesos-style.py:566:0: C0301: Line too long (104/80) (line-too-long)
> support/python3/mesos-style.py:155:0: W1401: Anomalous backslash in string: '\S'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
> support/python3/mesos-style.py:29:0: R0205: Class 'LinterBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/mesos-style.py:218:12: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
> support/python3/mesos-style.py:522:4: W0621: Redefining name 'build_virtualenv' from outer scope (line 512) (redefined-outer-name)
> ************* Module push-commits
> support/python3/push-commits.py:47:0: W0311: Bad indentation. Found 2 spaces, expected 4 (bad-indentation)
> support/python3/push-commits.py:54:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-list',
> ^| (bad-continuation)
> support/python3/push-commits.py:55:0: C0330: Wrong continued indentation (add 1 space).
> '--reverse',
> ^| (bad-continuation)
> support/python3/push-commits.py:56:0: C0330: Wrong continued indentation (add 1 space).
> revision_range]).strip().split('\n')
> ^| (bad-continuation)
> support/python3/push-commits.py:59:0: C0330: Wrong continued indentation (add 1 space).
> '--no-pager',
> ^| (bad-continuation)
> support/python3/push-commits.py:60:0: C0330: Wrong continued indentation (add 1 space).
> 'show',
> ^| (bad-continuation)
> support/python3/push-commits.py:61:0: C0330: Wrong continued indentation (add 1 space).
> '--no-color',
> ^| (bad-continuation)
> support/python3/push-commits.py:62:0: C0330: Wrong continued indentation (add 1 space).
> '--no-patch',
> ^| (bad-continuation)
> support/python3/push-commits.py:63:0: C0330: Wrong continued indentation (add 1 space).
> rev]).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:92:0: C0330: Wrong continued indentation (add 1 space).
> 'close',
> ^| (bad-continuation)
> support/python3/push-commits.py:93:0: C0330: Wrong continued indentation (add 1 space).
> '--description',
> ^| (bad-continuation)
> support/python3/push-commits.py:94:0: C0330: Wrong continued indentation (add 1 space).
> commit_log,
> ^| (bad-continuation)
> support/python3/push-commits.py:95:0: C0330: Wrong continued indentation (add 1 space).
> review_id])
> ^| (bad-continuation)
> support/python3/push-commits.py:127:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-parse',
> ^| (bad-continuation)
> support/python3/push-commits.py:128:0: C0330: Wrong continued indentation (add 1 space).
> '--abbrev-ref',
> ^| (bad-continuation)
> support/python3/push-commits.py:129:0: C0330: Wrong continued indentation (add 1 space).
> 'master@{upstream}']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:145:0: C0330: Wrong continued indentation (add 1 space).
> 'config',
> ^| (bad-continuation)
> support/python3/push-commits.py:146:0: C0330: Wrong continued indentation (add 1 space).
> '--get',
> ^| (bad-continuation)
> support/python3/push-commits.py:147:0: C0330: Wrong continued indentation (add 1 space).
> 'branch.master.remote']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:153:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:154:0: C0330: Wrong continued indentation (add 1 space).
> '--dry-run',
> ^| (bad-continuation)
> support/python3/push-commits.py:155:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:156:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> support/python3/push-commits.py:159:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:160:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:161:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> ```
> After:
> ```
> Checking 47 Python files
> Total errors found: 0
> ```
>
>
> Thanks,
>
> Armand Grillet
>
>
Re: Review Request 68608: Fixed all Python linting issues.
Posted by Mesos Reviewbot <re...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68608/#review208279
-----------------------------------------------------------
Patch looks great!
Reviews applied: [68608]
Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh
- Mesos Reviewbot
On Sept. 3, 2018, 11:33 a.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68608/
> -----------------------------------------------------------
>
> (Updated Sept. 3, 2018, 11:33 a.m.)
>
>
> Review request for mesos, Benjamin Bannier, Eric Chung, and Kevin Klues.
>
>
> Bugs: MESOS-9112
> https://issues.apache.org/jira/browse/MESOS-9112
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Fixed all Python linting issues.
>
>
> Diffs
> -----
>
> src/python/cli_new/bin/main.py b6f9faba741d7229a02cc5052e3b96c193df8991
> src/python/cli_new/bin/settings.py f501fefa2eb29ed5d7b55828dd18cb396e313583
> src/python/cli_new/lib/cli/config.py 92ff3eb96826651505e0c3e786f6292cef600bc7
> src/python/cli_new/lib/cli/docopt.py 4dedd7734bf5880c780be6ffe71430ac1d57aad3
> src/python/cli_new/lib/cli/plugins/__init__.py e743772afce3aad753be3cd2a923a4331bafc31c
> src/python/cli_new/lib/cli/plugins/agent/main.py 4ebf8cd63a52262615dbcaf1dd79ae2951778c84
> src/python/cli_new/lib/cli/plugins/base.py e01a7b2bc4d4cbabe706c8926913f43d2b4cf69c
> src/python/cli_new/lib/cli/plugins/task/main.py f9d6826560bcae724c8e47794a73b69b40b955ab
> src/python/cli_new/lib/cli/tests/__init__.py b7c825fc17aa6fc732b5933260dd17291126665a
> src/python/cli_new/lib/cli/tests/agent.py 6e5e56554b2a556410daeffaa6fbf219adee17a2
> src/python/cli_new/lib/cli/tests/base.py f58d616b59232786d3d9fb44d8dd91c5c3017568
> src/python/cli_new/lib/cli/tests/task.py 3a57094b1ad340dcd630c0e59783686432d8a6a5
> src/python/cli_new/lib/cli/util.py b7404318afd399b2cd54b33d6883b2e7c8734f85
> src/python/lib/mesos/http.py 8a5a3680b99aec5bebcb0f116e65ae820a72328f
> src/python/lib/tests/test_http.py c1b21dfcac0292c0646f16f1547ce949bea711a7
> support/apply-reviews.py 4e292a91f9eb3c6d4598ba258d6aae9dcfa86fc7
> support/generate-endpoint-help.py 37c4b58462ca42a42276ed1129190fc3359c01cf
> support/python3/apply-reviews.py 6e1d6d784ea1508c63ac7d5bb0f168a031e0db7c
> support/python3/common.py ed62c675fa20fb638b9e3aa63c9a55cae39fe9d7
> support/python3/generate-endpoint-help.py 6214d2efa41ca131ff47a52fbcef90fa245e1ed5
> support/python3/mesos-style.py 845ec73670961057d2d2d6ecceec99b0f087f2fa
> support/python3/push-commits.py 7d883505a535e9e3745dbc6dba795bcb6be9d938
> support/python3/test-upgrade.py a1745bd146d369da83f2a47c69d7f4d2c142dfa8
> support/verify-reviews.py c86db35c3231af8f1493519759a02cbaf7ef0209
>
>
> Diff: https://reviews.apache.org/r/68608/diff/2/
>
>
> Testing
> -------
>
> Before, `./support/mesos-style.py`:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> ************* Module verify-reviews
> C:261, 7: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
> ************* Module generate-endpoint-help
> R:215, 4: Unnecessary "else" after "return" (no-else-return)
> ************* 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)
> Total errors found: 5
> ```
> After:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> Total errors found: 0
> ```
>
> Before, `python3 ./support/python3/mesos-style.py`:
> ```
> Checking 47 Python files
> ************* Module cli.tests.agent
> lib/cli/tests/agent.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.plugins.task.main
> lib/cli/plugins/task/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.util
> lib/cli/util.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/util.py:282:0: R0205: Class 'Table' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.plugins.base
> lib/cli/plugins/base.py:119:0: E0012: Bad option value 'R0204' (bad-option-value)
> lib/cli/plugins/base.py:69:0: R0205: Class 'PluginBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> lib/cli/plugins/base.py:115:11: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module cli.tests.base
> lib/cli/tests/base.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/tests/base.py:69:8: R1705: Unnecessary "else" after "return" (no-else-return)
> lib/cli/tests/base.py:81:0: R0205: Class 'Executable' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module settings
> bin/settings.py:32:4: E0611: No name 'VERSION' in module 'version' (no-name-in-module)
> ************* Module cli.plugins.agent.main
> lib/cli/plugins/agent/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.config
> lib/cli/config.py:31:0: R0205: Class 'Config' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.tests.task
> lib/cli/tests/task.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module main
> bin/main.py:70:7: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module tests.test_http
> tests/test_http.py:265:8: R1705: Unnecessary "else" after "return" (no-else-return)
> tests/test_http.py:271:8: R1705: Unnecessary "else" after "return" (no-else-return)
> ************* Module mesos.http
> mesos/http.py:79:0: R0205: Class 'Resource' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module test-upgrade
> support/python3/test-upgrade.py:32:0: R0205: Class 'Process' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/test-upgrade.py:65:15: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module apply-reviews
> support/python3/apply-reviews.py:83:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> support/python3/apply-reviews.py:133:8: R1705: Unnecessary "else" after "return" (no-else-return)
> support/python3/apply-reviews.py:304:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> ************* Module common
> support/python3/common.py:38:0: R0205: Class 'ReviewBoardHandler' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module generate-endpoint-help
> support/python3/generate-endpoint-help.py:84:0: R0205: Class 'Subprocess' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module mesos-style
> support/python3/mesos-style.py:528:0: C0301: Line too long (84/80) (line-too-long)
> support/python3/mesos-style.py:551:0: C0301: Line too long (90/80) (line-too-long)
> support/python3/mesos-style.py:557:0: C0301: Line too long (86/80) (line-too-long)
> support/python3/mesos-style.py:566:0: C0301: Line too long (104/80) (line-too-long)
> support/python3/mesos-style.py:155:0: W1401: Anomalous backslash in string: '\S'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
> support/python3/mesos-style.py:29:0: R0205: Class 'LinterBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/mesos-style.py:218:12: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
> support/python3/mesos-style.py:522:4: W0621: Redefining name 'build_virtualenv' from outer scope (line 512) (redefined-outer-name)
> ************* Module push-commits
> support/python3/push-commits.py:47:0: W0311: Bad indentation. Found 2 spaces, expected 4 (bad-indentation)
> support/python3/push-commits.py:54:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-list',
> ^| (bad-continuation)
> support/python3/push-commits.py:55:0: C0330: Wrong continued indentation (add 1 space).
> '--reverse',
> ^| (bad-continuation)
> support/python3/push-commits.py:56:0: C0330: Wrong continued indentation (add 1 space).
> revision_range]).strip().split('\n')
> ^| (bad-continuation)
> support/python3/push-commits.py:59:0: C0330: Wrong continued indentation (add 1 space).
> '--no-pager',
> ^| (bad-continuation)
> support/python3/push-commits.py:60:0: C0330: Wrong continued indentation (add 1 space).
> 'show',
> ^| (bad-continuation)
> support/python3/push-commits.py:61:0: C0330: Wrong continued indentation (add 1 space).
> '--no-color',
> ^| (bad-continuation)
> support/python3/push-commits.py:62:0: C0330: Wrong continued indentation (add 1 space).
> '--no-patch',
> ^| (bad-continuation)
> support/python3/push-commits.py:63:0: C0330: Wrong continued indentation (add 1 space).
> rev]).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:92:0: C0330: Wrong continued indentation (add 1 space).
> 'close',
> ^| (bad-continuation)
> support/python3/push-commits.py:93:0: C0330: Wrong continued indentation (add 1 space).
> '--description',
> ^| (bad-continuation)
> support/python3/push-commits.py:94:0: C0330: Wrong continued indentation (add 1 space).
> commit_log,
> ^| (bad-continuation)
> support/python3/push-commits.py:95:0: C0330: Wrong continued indentation (add 1 space).
> review_id])
> ^| (bad-continuation)
> support/python3/push-commits.py:127:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-parse',
> ^| (bad-continuation)
> support/python3/push-commits.py:128:0: C0330: Wrong continued indentation (add 1 space).
> '--abbrev-ref',
> ^| (bad-continuation)
> support/python3/push-commits.py:129:0: C0330: Wrong continued indentation (add 1 space).
> 'master@{upstream}']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:145:0: C0330: Wrong continued indentation (add 1 space).
> 'config',
> ^| (bad-continuation)
> support/python3/push-commits.py:146:0: C0330: Wrong continued indentation (add 1 space).
> '--get',
> ^| (bad-continuation)
> support/python3/push-commits.py:147:0: C0330: Wrong continued indentation (add 1 space).
> 'branch.master.remote']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:153:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:154:0: C0330: Wrong continued indentation (add 1 space).
> '--dry-run',
> ^| (bad-continuation)
> support/python3/push-commits.py:155:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:156:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> support/python3/push-commits.py:159:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:160:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:161:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> ```
> After:
> ```
> Checking 47 Python files
> Total errors found: 0
> ```
>
>
> Thanks,
>
> Armand Grillet
>
>
Re: Review Request 68608: Fixed all Python linting issues.
Posted by Benjamin Bannier <be...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68608/#review208293
-----------------------------------------------------------
Ship it!
Ship It!
- Benjamin Bannier
On Sept. 3, 2018, 5:33 p.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68608/
> -----------------------------------------------------------
>
> (Updated Sept. 3, 2018, 5:33 p.m.)
>
>
> Review request for mesos, Benjamin Bannier, Eric Chung, and Kevin Klues.
>
>
> Bugs: MESOS-9112
> https://issues.apache.org/jira/browse/MESOS-9112
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Fixed all Python linting issues.
>
>
> Diffs
> -----
>
> src/python/cli_new/bin/main.py b6f9faba741d7229a02cc5052e3b96c193df8991
> src/python/cli_new/bin/settings.py f501fefa2eb29ed5d7b55828dd18cb396e313583
> src/python/cli_new/lib/cli/config.py 92ff3eb96826651505e0c3e786f6292cef600bc7
> src/python/cli_new/lib/cli/docopt.py 4dedd7734bf5880c780be6ffe71430ac1d57aad3
> src/python/cli_new/lib/cli/plugins/__init__.py e743772afce3aad753be3cd2a923a4331bafc31c
> src/python/cli_new/lib/cli/plugins/agent/main.py 4ebf8cd63a52262615dbcaf1dd79ae2951778c84
> src/python/cli_new/lib/cli/plugins/base.py e01a7b2bc4d4cbabe706c8926913f43d2b4cf69c
> src/python/cli_new/lib/cli/plugins/task/main.py f9d6826560bcae724c8e47794a73b69b40b955ab
> src/python/cli_new/lib/cli/tests/__init__.py b7c825fc17aa6fc732b5933260dd17291126665a
> src/python/cli_new/lib/cli/tests/agent.py 6e5e56554b2a556410daeffaa6fbf219adee17a2
> src/python/cli_new/lib/cli/tests/base.py f58d616b59232786d3d9fb44d8dd91c5c3017568
> src/python/cli_new/lib/cli/tests/task.py 3a57094b1ad340dcd630c0e59783686432d8a6a5
> src/python/cli_new/lib/cli/util.py b7404318afd399b2cd54b33d6883b2e7c8734f85
> src/python/lib/mesos/http.py 8a5a3680b99aec5bebcb0f116e65ae820a72328f
> src/python/lib/tests/test_http.py c1b21dfcac0292c0646f16f1547ce949bea711a7
> support/apply-reviews.py 4e292a91f9eb3c6d4598ba258d6aae9dcfa86fc7
> support/generate-endpoint-help.py 37c4b58462ca42a42276ed1129190fc3359c01cf
> support/python3/apply-reviews.py 6e1d6d784ea1508c63ac7d5bb0f168a031e0db7c
> support/python3/common.py ed62c675fa20fb638b9e3aa63c9a55cae39fe9d7
> support/python3/generate-endpoint-help.py 6214d2efa41ca131ff47a52fbcef90fa245e1ed5
> support/python3/mesos-style.py 845ec73670961057d2d2d6ecceec99b0f087f2fa
> support/python3/push-commits.py 7d883505a535e9e3745dbc6dba795bcb6be9d938
> support/python3/test-upgrade.py a1745bd146d369da83f2a47c69d7f4d2c142dfa8
> support/verify-reviews.py c86db35c3231af8f1493519759a02cbaf7ef0209
>
>
> Diff: https://reviews.apache.org/r/68608/diff/2/
>
>
> Testing
> -------
>
> Before, `./support/mesos-style.py`:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> ************* Module verify-reviews
> C:261, 7: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
> ************* Module generate-endpoint-help
> R:215, 4: Unnecessary "else" after "return" (no-else-return)
> ************* 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)
> Total errors found: 5
> ```
> After:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> Total errors found: 0
> ```
>
> Before, `python3 ./support/python3/mesos-style.py`:
> ```
> Checking 47 Python files
> ************* Module cli.tests.agent
> lib/cli/tests/agent.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.plugins.task.main
> lib/cli/plugins/task/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.util
> lib/cli/util.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/util.py:282:0: R0205: Class 'Table' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.plugins.base
> lib/cli/plugins/base.py:119:0: E0012: Bad option value 'R0204' (bad-option-value)
> lib/cli/plugins/base.py:69:0: R0205: Class 'PluginBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> lib/cli/plugins/base.py:115:11: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module cli.tests.base
> lib/cli/tests/base.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/tests/base.py:69:8: R1705: Unnecessary "else" after "return" (no-else-return)
> lib/cli/tests/base.py:81:0: R0205: Class 'Executable' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module settings
> bin/settings.py:32:4: E0611: No name 'VERSION' in module 'version' (no-name-in-module)
> ************* Module cli.plugins.agent.main
> lib/cli/plugins/agent/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.config
> lib/cli/config.py:31:0: R0205: Class 'Config' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.tests.task
> lib/cli/tests/task.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module main
> bin/main.py:70:7: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module tests.test_http
> tests/test_http.py:265:8: R1705: Unnecessary "else" after "return" (no-else-return)
> tests/test_http.py:271:8: R1705: Unnecessary "else" after "return" (no-else-return)
> ************* Module mesos.http
> mesos/http.py:79:0: R0205: Class 'Resource' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module test-upgrade
> support/python3/test-upgrade.py:32:0: R0205: Class 'Process' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/test-upgrade.py:65:15: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module apply-reviews
> support/python3/apply-reviews.py:83:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> support/python3/apply-reviews.py:133:8: R1705: Unnecessary "else" after "return" (no-else-return)
> support/python3/apply-reviews.py:304:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> ************* Module common
> support/python3/common.py:38:0: R0205: Class 'ReviewBoardHandler' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module generate-endpoint-help
> support/python3/generate-endpoint-help.py:84:0: R0205: Class 'Subprocess' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module mesos-style
> support/python3/mesos-style.py:528:0: C0301: Line too long (84/80) (line-too-long)
> support/python3/mesos-style.py:551:0: C0301: Line too long (90/80) (line-too-long)
> support/python3/mesos-style.py:557:0: C0301: Line too long (86/80) (line-too-long)
> support/python3/mesos-style.py:566:0: C0301: Line too long (104/80) (line-too-long)
> support/python3/mesos-style.py:155:0: W1401: Anomalous backslash in string: '\S'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
> support/python3/mesos-style.py:29:0: R0205: Class 'LinterBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/mesos-style.py:218:12: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
> support/python3/mesos-style.py:522:4: W0621: Redefining name 'build_virtualenv' from outer scope (line 512) (redefined-outer-name)
> ************* Module push-commits
> support/python3/push-commits.py:47:0: W0311: Bad indentation. Found 2 spaces, expected 4 (bad-indentation)
> support/python3/push-commits.py:54:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-list',
> ^| (bad-continuation)
> support/python3/push-commits.py:55:0: C0330: Wrong continued indentation (add 1 space).
> '--reverse',
> ^| (bad-continuation)
> support/python3/push-commits.py:56:0: C0330: Wrong continued indentation (add 1 space).
> revision_range]).strip().split('\n')
> ^| (bad-continuation)
> support/python3/push-commits.py:59:0: C0330: Wrong continued indentation (add 1 space).
> '--no-pager',
> ^| (bad-continuation)
> support/python3/push-commits.py:60:0: C0330: Wrong continued indentation (add 1 space).
> 'show',
> ^| (bad-continuation)
> support/python3/push-commits.py:61:0: C0330: Wrong continued indentation (add 1 space).
> '--no-color',
> ^| (bad-continuation)
> support/python3/push-commits.py:62:0: C0330: Wrong continued indentation (add 1 space).
> '--no-patch',
> ^| (bad-continuation)
> support/python3/push-commits.py:63:0: C0330: Wrong continued indentation (add 1 space).
> rev]).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:92:0: C0330: Wrong continued indentation (add 1 space).
> 'close',
> ^| (bad-continuation)
> support/python3/push-commits.py:93:0: C0330: Wrong continued indentation (add 1 space).
> '--description',
> ^| (bad-continuation)
> support/python3/push-commits.py:94:0: C0330: Wrong continued indentation (add 1 space).
> commit_log,
> ^| (bad-continuation)
> support/python3/push-commits.py:95:0: C0330: Wrong continued indentation (add 1 space).
> review_id])
> ^| (bad-continuation)
> support/python3/push-commits.py:127:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-parse',
> ^| (bad-continuation)
> support/python3/push-commits.py:128:0: C0330: Wrong continued indentation (add 1 space).
> '--abbrev-ref',
> ^| (bad-continuation)
> support/python3/push-commits.py:129:0: C0330: Wrong continued indentation (add 1 space).
> 'master@{upstream}']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:145:0: C0330: Wrong continued indentation (add 1 space).
> 'config',
> ^| (bad-continuation)
> support/python3/push-commits.py:146:0: C0330: Wrong continued indentation (add 1 space).
> '--get',
> ^| (bad-continuation)
> support/python3/push-commits.py:147:0: C0330: Wrong continued indentation (add 1 space).
> 'branch.master.remote']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:153:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:154:0: C0330: Wrong continued indentation (add 1 space).
> '--dry-run',
> ^| (bad-continuation)
> support/python3/push-commits.py:155:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:156:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> support/python3/push-commits.py:159:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:160:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:161:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> ```
> After:
> ```
> Checking 47 Python files
> Total errors found: 0
> ```
>
>
> Thanks,
>
> Armand Grillet
>
>
Re: Review Request 68608: Fixed all Python linting issues.
Posted by Benno Evers <be...@mesosphere.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68608/#review208266
-----------------------------------------------------------
Ship it!
Ship It!
- Benno Evers
On Sept. 3, 2018, 3:33 p.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68608/
> -----------------------------------------------------------
>
> (Updated Sept. 3, 2018, 3:33 p.m.)
>
>
> Review request for mesos, Benjamin Bannier, Eric Chung, and Kevin Klues.
>
>
> Bugs: MESOS-9112
> https://issues.apache.org/jira/browse/MESOS-9112
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Fixed all Python linting issues.
>
>
> Diffs
> -----
>
> src/python/cli_new/bin/main.py b6f9faba741d7229a02cc5052e3b96c193df8991
> src/python/cli_new/bin/settings.py f501fefa2eb29ed5d7b55828dd18cb396e313583
> src/python/cli_new/lib/cli/config.py 92ff3eb96826651505e0c3e786f6292cef600bc7
> src/python/cli_new/lib/cli/docopt.py 4dedd7734bf5880c780be6ffe71430ac1d57aad3
> src/python/cli_new/lib/cli/plugins/__init__.py e743772afce3aad753be3cd2a923a4331bafc31c
> src/python/cli_new/lib/cli/plugins/agent/main.py 4ebf8cd63a52262615dbcaf1dd79ae2951778c84
> src/python/cli_new/lib/cli/plugins/base.py e01a7b2bc4d4cbabe706c8926913f43d2b4cf69c
> src/python/cli_new/lib/cli/plugins/task/main.py f9d6826560bcae724c8e47794a73b69b40b955ab
> src/python/cli_new/lib/cli/tests/__init__.py b7c825fc17aa6fc732b5933260dd17291126665a
> src/python/cli_new/lib/cli/tests/agent.py 6e5e56554b2a556410daeffaa6fbf219adee17a2
> src/python/cli_new/lib/cli/tests/base.py f58d616b59232786d3d9fb44d8dd91c5c3017568
> src/python/cli_new/lib/cli/tests/task.py 3a57094b1ad340dcd630c0e59783686432d8a6a5
> src/python/cli_new/lib/cli/util.py b7404318afd399b2cd54b33d6883b2e7c8734f85
> src/python/lib/mesos/http.py 8a5a3680b99aec5bebcb0f116e65ae820a72328f
> src/python/lib/tests/test_http.py c1b21dfcac0292c0646f16f1547ce949bea711a7
> support/apply-reviews.py 4e292a91f9eb3c6d4598ba258d6aae9dcfa86fc7
> support/generate-endpoint-help.py 37c4b58462ca42a42276ed1129190fc3359c01cf
> support/python3/apply-reviews.py 6e1d6d784ea1508c63ac7d5bb0f168a031e0db7c
> support/python3/common.py ed62c675fa20fb638b9e3aa63c9a55cae39fe9d7
> support/python3/generate-endpoint-help.py 6214d2efa41ca131ff47a52fbcef90fa245e1ed5
> support/python3/mesos-style.py 845ec73670961057d2d2d6ecceec99b0f087f2fa
> support/python3/push-commits.py 7d883505a535e9e3745dbc6dba795bcb6be9d938
> support/python3/test-upgrade.py a1745bd146d369da83f2a47c69d7f4d2c142dfa8
> support/verify-reviews.py c86db35c3231af8f1493519759a02cbaf7ef0209
>
>
> Diff: https://reviews.apache.org/r/68608/diff/2/
>
>
> Testing
> -------
>
> Before, `./support/mesos-style.py`:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> ************* Module verify-reviews
> C:261, 7: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
> ************* Module generate-endpoint-help
> R:215, 4: Unnecessary "else" after "return" (no-else-return)
> ************* 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)
> Total errors found: 5
> ```
> After:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> Total errors found: 0
> ```
>
> Before, `python3 ./support/python3/mesos-style.py`:
> ```
> Checking 47 Python files
> ************* Module cli.tests.agent
> lib/cli/tests/agent.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.plugins.task.main
> lib/cli/plugins/task/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.util
> lib/cli/util.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/util.py:282:0: R0205: Class 'Table' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.plugins.base
> lib/cli/plugins/base.py:119:0: E0012: Bad option value 'R0204' (bad-option-value)
> lib/cli/plugins/base.py:69:0: R0205: Class 'PluginBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> lib/cli/plugins/base.py:115:11: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module cli.tests.base
> lib/cli/tests/base.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/tests/base.py:69:8: R1705: Unnecessary "else" after "return" (no-else-return)
> lib/cli/tests/base.py:81:0: R0205: Class 'Executable' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module settings
> bin/settings.py:32:4: E0611: No name 'VERSION' in module 'version' (no-name-in-module)
> ************* Module cli.plugins.agent.main
> lib/cli/plugins/agent/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.config
> lib/cli/config.py:31:0: R0205: Class 'Config' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.tests.task
> lib/cli/tests/task.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module main
> bin/main.py:70:7: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module tests.test_http
> tests/test_http.py:265:8: R1705: Unnecessary "else" after "return" (no-else-return)
> tests/test_http.py:271:8: R1705: Unnecessary "else" after "return" (no-else-return)
> ************* Module mesos.http
> mesos/http.py:79:0: R0205: Class 'Resource' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module test-upgrade
> support/python3/test-upgrade.py:32:0: R0205: Class 'Process' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/test-upgrade.py:65:15: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module apply-reviews
> support/python3/apply-reviews.py:83:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> support/python3/apply-reviews.py:133:8: R1705: Unnecessary "else" after "return" (no-else-return)
> support/python3/apply-reviews.py:304:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> ************* Module common
> support/python3/common.py:38:0: R0205: Class 'ReviewBoardHandler' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module generate-endpoint-help
> support/python3/generate-endpoint-help.py:84:0: R0205: Class 'Subprocess' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module mesos-style
> support/python3/mesos-style.py:528:0: C0301: Line too long (84/80) (line-too-long)
> support/python3/mesos-style.py:551:0: C0301: Line too long (90/80) (line-too-long)
> support/python3/mesos-style.py:557:0: C0301: Line too long (86/80) (line-too-long)
> support/python3/mesos-style.py:566:0: C0301: Line too long (104/80) (line-too-long)
> support/python3/mesos-style.py:155:0: W1401: Anomalous backslash in string: '\S'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
> support/python3/mesos-style.py:29:0: R0205: Class 'LinterBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/mesos-style.py:218:12: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
> support/python3/mesos-style.py:522:4: W0621: Redefining name 'build_virtualenv' from outer scope (line 512) (redefined-outer-name)
> ************* Module push-commits
> support/python3/push-commits.py:47:0: W0311: Bad indentation. Found 2 spaces, expected 4 (bad-indentation)
> support/python3/push-commits.py:54:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-list',
> ^| (bad-continuation)
> support/python3/push-commits.py:55:0: C0330: Wrong continued indentation (add 1 space).
> '--reverse',
> ^| (bad-continuation)
> support/python3/push-commits.py:56:0: C0330: Wrong continued indentation (add 1 space).
> revision_range]).strip().split('\n')
> ^| (bad-continuation)
> support/python3/push-commits.py:59:0: C0330: Wrong continued indentation (add 1 space).
> '--no-pager',
> ^| (bad-continuation)
> support/python3/push-commits.py:60:0: C0330: Wrong continued indentation (add 1 space).
> 'show',
> ^| (bad-continuation)
> support/python3/push-commits.py:61:0: C0330: Wrong continued indentation (add 1 space).
> '--no-color',
> ^| (bad-continuation)
> support/python3/push-commits.py:62:0: C0330: Wrong continued indentation (add 1 space).
> '--no-patch',
> ^| (bad-continuation)
> support/python3/push-commits.py:63:0: C0330: Wrong continued indentation (add 1 space).
> rev]).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:92:0: C0330: Wrong continued indentation (add 1 space).
> 'close',
> ^| (bad-continuation)
> support/python3/push-commits.py:93:0: C0330: Wrong continued indentation (add 1 space).
> '--description',
> ^| (bad-continuation)
> support/python3/push-commits.py:94:0: C0330: Wrong continued indentation (add 1 space).
> commit_log,
> ^| (bad-continuation)
> support/python3/push-commits.py:95:0: C0330: Wrong continued indentation (add 1 space).
> review_id])
> ^| (bad-continuation)
> support/python3/push-commits.py:127:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-parse',
> ^| (bad-continuation)
> support/python3/push-commits.py:128:0: C0330: Wrong continued indentation (add 1 space).
> '--abbrev-ref',
> ^| (bad-continuation)
> support/python3/push-commits.py:129:0: C0330: Wrong continued indentation (add 1 space).
> 'master@{upstream}']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:145:0: C0330: Wrong continued indentation (add 1 space).
> 'config',
> ^| (bad-continuation)
> support/python3/push-commits.py:146:0: C0330: Wrong continued indentation (add 1 space).
> '--get',
> ^| (bad-continuation)
> support/python3/push-commits.py:147:0: C0330: Wrong continued indentation (add 1 space).
> 'branch.master.remote']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:153:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:154:0: C0330: Wrong continued indentation (add 1 space).
> '--dry-run',
> ^| (bad-continuation)
> support/python3/push-commits.py:155:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:156:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> support/python3/push-commits.py:159:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:160:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:161:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> ```
> After:
> ```
> Checking 47 Python files
> Total errors found: 0
> ```
>
>
> Thanks,
>
> Armand Grillet
>
>
Re: Review Request 68608: Fixed all Python linting issues.
Posted by Mesos Reviewbot Windows <re...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68608/#review208269
-----------------------------------------------------------
PASS: Mesos patch 68608 was successfully built and tested.
Reviews applied: `['68608']`
All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2283/mesos-review-68608
- Mesos Reviewbot Windows
On Sept. 3, 2018, 8:33 a.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68608/
> -----------------------------------------------------------
>
> (Updated Sept. 3, 2018, 8:33 a.m.)
>
>
> Review request for mesos, Benjamin Bannier, Eric Chung, and Kevin Klues.
>
>
> Bugs: MESOS-9112
> https://issues.apache.org/jira/browse/MESOS-9112
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Fixed all Python linting issues.
>
>
> Diffs
> -----
>
> src/python/cli_new/bin/main.py b6f9faba741d7229a02cc5052e3b96c193df8991
> src/python/cli_new/bin/settings.py f501fefa2eb29ed5d7b55828dd18cb396e313583
> src/python/cli_new/lib/cli/config.py 92ff3eb96826651505e0c3e786f6292cef600bc7
> src/python/cli_new/lib/cli/docopt.py 4dedd7734bf5880c780be6ffe71430ac1d57aad3
> src/python/cli_new/lib/cli/plugins/__init__.py e743772afce3aad753be3cd2a923a4331bafc31c
> src/python/cli_new/lib/cli/plugins/agent/main.py 4ebf8cd63a52262615dbcaf1dd79ae2951778c84
> src/python/cli_new/lib/cli/plugins/base.py e01a7b2bc4d4cbabe706c8926913f43d2b4cf69c
> src/python/cli_new/lib/cli/plugins/task/main.py f9d6826560bcae724c8e47794a73b69b40b955ab
> src/python/cli_new/lib/cli/tests/__init__.py b7c825fc17aa6fc732b5933260dd17291126665a
> src/python/cli_new/lib/cli/tests/agent.py 6e5e56554b2a556410daeffaa6fbf219adee17a2
> src/python/cli_new/lib/cli/tests/base.py f58d616b59232786d3d9fb44d8dd91c5c3017568
> src/python/cli_new/lib/cli/tests/task.py 3a57094b1ad340dcd630c0e59783686432d8a6a5
> src/python/cli_new/lib/cli/util.py b7404318afd399b2cd54b33d6883b2e7c8734f85
> src/python/lib/mesos/http.py 8a5a3680b99aec5bebcb0f116e65ae820a72328f
> src/python/lib/tests/test_http.py c1b21dfcac0292c0646f16f1547ce949bea711a7
> support/apply-reviews.py 4e292a91f9eb3c6d4598ba258d6aae9dcfa86fc7
> support/generate-endpoint-help.py 37c4b58462ca42a42276ed1129190fc3359c01cf
> support/python3/apply-reviews.py 6e1d6d784ea1508c63ac7d5bb0f168a031e0db7c
> support/python3/common.py ed62c675fa20fb638b9e3aa63c9a55cae39fe9d7
> support/python3/generate-endpoint-help.py 6214d2efa41ca131ff47a52fbcef90fa245e1ed5
> support/python3/mesos-style.py 845ec73670961057d2d2d6ecceec99b0f087f2fa
> support/python3/push-commits.py 7d883505a535e9e3745dbc6dba795bcb6be9d938
> support/python3/test-upgrade.py a1745bd146d369da83f2a47c69d7f4d2c142dfa8
> support/verify-reviews.py c86db35c3231af8f1493519759a02cbaf7ef0209
>
>
> Diff: https://reviews.apache.org/r/68608/diff/2/
>
>
> Testing
> -------
>
> Before, `./support/mesos-style.py`:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> ************* Module verify-reviews
> C:261, 7: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
> ************* Module generate-endpoint-help
> R:215, 4: Unnecessary "else" after "return" (no-else-return)
> ************* 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)
> Total errors found: 5
> ```
> After:
> ```
> Checking 12 Python files
> Using config file /Users/Armand/Code/apache-mesos/support/pylint.config
> Total errors found: 0
> ```
>
> Before, `python3 ./support/python3/mesos-style.py`:
> ```
> Checking 47 Python files
> ************* Module cli.tests.agent
> lib/cli/tests/agent.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.plugins.task.main
> lib/cli/plugins/task/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.util
> lib/cli/util.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/util.py:282:0: R0205: Class 'Table' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.plugins.base
> lib/cli/plugins/base.py:119:0: E0012: Bad option value 'R0204' (bad-option-value)
> lib/cli/plugins/base.py:69:0: R0205: Class 'PluginBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> lib/cli/plugins/base.py:115:11: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module cli.tests.base
> lib/cli/tests/base.py:31:0: C0414: Import alias does not rename original package (useless-import-alias)
> lib/cli/tests/base.py:69:8: R1705: Unnecessary "else" after "return" (no-else-return)
> lib/cli/tests/base.py:81:0: R0205: Class 'Executable' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module settings
> bin/settings.py:32:4: E0611: No name 'VERSION' in module 'version' (no-name-in-module)
> ************* Module cli.plugins.agent.main
> lib/cli/plugins/agent/main.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module cli.config
> lib/cli/config.py:31:0: R0205: Class 'Config' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module cli.tests.task
> lib/cli/tests/task.py:21:0: C0414: Import alias does not rename original package (useless-import-alias)
> ************* Module main
> bin/main.py:70:7: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module tests.test_http
> tests/test_http.py:265:8: R1705: Unnecessary "else" after "return" (no-else-return)
> tests/test_http.py:271:8: R1705: Unnecessary "else" after "return" (no-else-return)
> ************* Module mesos.http
> mesos/http.py:79:0: R0205: Class 'Resource' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module test-upgrade
> support/python3/test-upgrade.py:32:0: R0205: Class 'Process' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/test-upgrade.py:65:15: C0121: Comparison to None should be 'expr is not None' (singleton-comparison)
> ************* Module apply-reviews
> support/python3/apply-reviews.py:83:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> support/python3/apply-reviews.py:133:8: R1705: Unnecessary "else" after "return" (no-else-return)
> support/python3/apply-reviews.py:304:4: R1705: Unnecessary "elif" after "return" (no-else-return)
> ************* Module common
> support/python3/common.py:38:0: R0205: Class 'ReviewBoardHandler' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module generate-endpoint-help
> support/python3/generate-endpoint-help.py:84:0: R0205: Class 'Subprocess' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> ************* Module mesos-style
> support/python3/mesos-style.py:528:0: C0301: Line too long (84/80) (line-too-long)
> support/python3/mesos-style.py:551:0: C0301: Line too long (90/80) (line-too-long)
> support/python3/mesos-style.py:557:0: C0301: Line too long (86/80) (line-too-long)
> support/python3/mesos-style.py:566:0: C0301: Line too long (104/80) (line-too-long)
> support/python3/mesos-style.py:155:0: W1401: Anomalous backslash in string: '\S'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
> support/python3/mesos-style.py:29:0: R0205: Class 'LinterBase' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
> support/python3/mesos-style.py:218:12: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
> support/python3/mesos-style.py:522:4: W0621: Redefining name 'build_virtualenv' from outer scope (line 512) (redefined-outer-name)
> ************* Module push-commits
> support/python3/push-commits.py:47:0: W0311: Bad indentation. Found 2 spaces, expected 4 (bad-indentation)
> support/python3/push-commits.py:54:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-list',
> ^| (bad-continuation)
> support/python3/push-commits.py:55:0: C0330: Wrong continued indentation (add 1 space).
> '--reverse',
> ^| (bad-continuation)
> support/python3/push-commits.py:56:0: C0330: Wrong continued indentation (add 1 space).
> revision_range]).strip().split('\n')
> ^| (bad-continuation)
> support/python3/push-commits.py:59:0: C0330: Wrong continued indentation (add 1 space).
> '--no-pager',
> ^| (bad-continuation)
> support/python3/push-commits.py:60:0: C0330: Wrong continued indentation (add 1 space).
> 'show',
> ^| (bad-continuation)
> support/python3/push-commits.py:61:0: C0330: Wrong continued indentation (add 1 space).
> '--no-color',
> ^| (bad-continuation)
> support/python3/push-commits.py:62:0: C0330: Wrong continued indentation (add 1 space).
> '--no-patch',
> ^| (bad-continuation)
> support/python3/push-commits.py:63:0: C0330: Wrong continued indentation (add 1 space).
> rev]).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:92:0: C0330: Wrong continued indentation (add 1 space).
> 'close',
> ^| (bad-continuation)
> support/python3/push-commits.py:93:0: C0330: Wrong continued indentation (add 1 space).
> '--description',
> ^| (bad-continuation)
> support/python3/push-commits.py:94:0: C0330: Wrong continued indentation (add 1 space).
> commit_log,
> ^| (bad-continuation)
> support/python3/push-commits.py:95:0: C0330: Wrong continued indentation (add 1 space).
> review_id])
> ^| (bad-continuation)
> support/python3/push-commits.py:127:0: C0330: Wrong continued indentation (add 1 space).
> 'rev-parse',
> ^| (bad-continuation)
> support/python3/push-commits.py:128:0: C0330: Wrong continued indentation (add 1 space).
> '--abbrev-ref',
> ^| (bad-continuation)
> support/python3/push-commits.py:129:0: C0330: Wrong continued indentation (add 1 space).
> 'master@{upstream}']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:145:0: C0330: Wrong continued indentation (add 1 space).
> 'config',
> ^| (bad-continuation)
> support/python3/push-commits.py:146:0: C0330: Wrong continued indentation (add 1 space).
> '--get',
> ^| (bad-continuation)
> support/python3/push-commits.py:147:0: C0330: Wrong continued indentation (add 1 space).
> 'branch.master.remote']).strip()
> ^| (bad-continuation)
> support/python3/push-commits.py:153:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:154:0: C0330: Wrong continued indentation (add 1 space).
> '--dry-run',
> ^| (bad-continuation)
> support/python3/push-commits.py:155:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:156:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> support/python3/push-commits.py:159:0: C0330: Wrong continued indentation (add 1 space).
> 'push',
> ^| (bad-continuation)
> support/python3/push-commits.py:160:0: C0330: Wrong continued indentation (add 1 space).
> remote,
> ^| (bad-continuation)
> support/python3/push-commits.py:161:0: C0330: Wrong continued indentation (add 1 space).
> 'master:master'])
> ^| (bad-continuation)
> ```
> After:
> ```
> Checking 47 Python files
> Total errors found: 0
> ```
>
>
> Thanks,
>
> Armand Grillet
>
>