[ 
https://issues.apache.org/jira/browse/CASSANDRA-14076?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16273643#comment-16273643
 ] 

Jay Zhuang edited comment on CASSANDRA-14076 at 11/30/17 11:32 PM:
-------------------------------------------------------------------

Here is the patch, please review:
| Branch | TravisCI Build Status |
| [14076|https://github.com/cooldoger/cassandra-dtest/tree/14076] | 
[!https://travis-ci.org/cooldoger/cassandra-dtest.svg?branch=14076!|https://travis-ci.org/cooldoger/cassandra-dtest/builds/309766256]
 |


was (Author: jay.zhuang):
Here is the patch, please review:
| Branch | TravisCI Build Status |
| [14076|https://github.com/cooldoger/cassandra/tree/14076] | 
[!https://travis-ci.org/cooldoger/cassandra-dtest.svg?branch=14076!|https://travis-ci.org/cooldoger/cassandra-dtest/builds/309766256]
 |

> dtest code style check failed
> -----------------------------
>
>                 Key: CASSANDRA-14076
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-14076
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Testing
>            Reporter: Jay Zhuang
>            Assignee: Jay Zhuang
>
> https://travis-ci.org/cooldoger/cassandra-dtest
> {noformat}
> $ flake8 --ignore=E501,F811,F812,F822,F823,F831,F841,N8,C9 
> --exclude=thrift_bindings,cassandra-thrift .
> ./consistency_test.py:547:17: E722 do not use bare except'
> ./consistency_test.py:976:49: E251 unexpected spaces around keyword / 
> parameter equals
> ./consistency_test.py:976:51: E251 unexpected spaces around keyword / 
> parameter equals
> ./consistency_test.py:981:63: E703 statement ends with a semicolon
> ./consistency_test.py:1037:49: E251 unexpected spaces around keyword / 
> parameter equals
> ./consistency_test.py:1037:51: E251 unexpected spaces around keyword / 
> parameter equals
> ./consistency_test.py:1054:46: E261 at least two spaces before inline comment
> ./consistency_test.py:1103:22: E251 unexpected spaces around keyword / 
> parameter equals
> ./consistency_test.py:1103:24: E251 unexpected spaces around keyword / 
> parameter equals
> ./consistency_test.py:1175:22: E251 unexpected spaces around keyword / 
> parameter equals
> ./consistency_test.py:1175:24: E251 unexpected spaces around keyword / 
> parameter equals
> ./counter_tests.py:59:24: E703 statement ends with a semicolon
> ./counter_tests.py:383:37: E261 at least two spaces before inline comment
> ./dtest.py:586:13: E722 do not use bare except'
> ./dtest.py:1130:1: E302 expected 2 blank lines, found 1
> ./nodetool_test.py:9:1: E302 expected 2 blank lines, found 1
> ./nodetool_test.py:78:1: W293 blank line contains whitespace
> ./nodetool_test.py:174:45: E261 at least two spaces before inline comment
> ./run_dtests.py:220:54: E221 multiple spaces before operator
> ./secondary_indexes_test.py:14:1: F401 'dtest.DtestTimeoutError' imported but 
> unused
> ./secondary_indexes_test.py:17:1: F401 'tools.data.index_is_built' imported 
> but unused
> ./secondary_indexes_test.py:21:1: E302 expected 2 blank lines, found 1
> ./sslnodetonode_test.py:15:1: E302 expected 2 blank lines, found 1
> ./sslnodetonode_test.py:191:1: W293 blank line contains whitespace
> ./sslnodetonode_test.py:191:1: W391 blank line at end of file
> ./system_keyspaces_test.py:6:1: E302 expected 2 blank lines, found 1
> ./system_keyspaces_test.py:28:59: E241 multiple spaces after ','
> ./system_keyspaces_test.py:50:62: E241 multiple spaces after ','
> ./write_failures_test.py:5:1: F401 'distutils.version.LooseVersion' imported 
> but unused
> ./plugins/dtestcollect.py:1:1: F401 'collections.namedtuple' imported but 
> unused
> ./plugins/dtestcollect.py:3:1: F401 'pprint.pprint' imported but unused
> ./plugins/dtestcollect.py:5:1: F401 'inspect' imported but unused
> ./plugins/dtestcollect.py:13:1: E302 expected 2 blank lines, found 1
> ./plugins/dtestcollect.py:44:9: E306 expected 1 blank line before a nested 
> definition, found 0
> ./plugins/dtestcollect.py:62:22: E703 statement ends with a semicolon
> ./plugins/dtestcollect.py:64:1: E302 expected 2 blank lines, found 1
> ./plugins/dtesttag.py:1:1: F401 'collections.namedtuple' imported but unused
> ./plugins/dtesttag.py:4:1: F401 'pprint.pprint' imported but unused
> ./plugins/dtesttag.py:8:1: E302 expected 2 blank lines, found 1
> ./plugins/dtesttag.py:20:1: W293 blank line contains whitespace
> ./plugins/dtesttag.py:25:1: W293 blank line contains whitespace
> ./plugins/dtestxunit.py:43:1: F401 'doctest' imported but unused
> ./plugins/dtestxunit.py:46:1: F401 'traceback' imported but unused
> ./plugins/dtestxunit.py:62:1: E302 expected 2 blank lines, found 1
> ./plugins/dtestxunit.py:66:1: E302 expected 2 blank lines, found 1
> ./plugins/dtestxunit.py:70:1: E302 expected 2 blank lines, found 1
> ./plugins/dtestxunit.py:76:29: E226 missing whitespace around arithmetic 
> operator
> ./plugins/dtestxunit.py:84:1: E302 expected 2 blank lines, found 1
> ./plugins/dtestxunit.py:107:1: E302 expected 2 blank lines, found 1
> ./plugins/dtestxunit.py:126:1: E302 expected 2 blank lines, found 1
> ./plugins/dtestxunit.py:219:32: W503 line break before binary operator
> ./plugins/dtestxunit.py:269:25: E126 continuation line over-indented for 
> hanging indent
> ./plugins/dtestxunit.py:277:25: E126 continuation line over-indented for 
> hanging indent
> ./repair_tests/deprecated_repair_test.py:159:9: E741 ambiguous variable name 
> 'l'
> ./repair_tests/incremental_repair_test.py:772:4: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:773:76: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:774:69: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:804:4: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:805:87: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:806:73: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:830:4: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:831:87: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:832:73: W291 trailing whitespace
> ./repair_tests/incremental_repair_test.py:855:55: E231 missing whitespace 
> after ','
> ./repair_tests/incremental_repair_test.py:855:57: E231 missing whitespace 
> after ','
> ./repair_tests/incremental_repair_test.py:855:59: E231 missing whitespace 
> after ','
> ./tools/data.py:154:1: E302 expected 2 blank lines, found 1
> ./tools/data.py:167:34: W292 no newline at end of file
> ./upgrade_tests/thrift_upgrade_test.py:25:65: E261 at least two spaces before 
> inline comment
> ./upgrade_tests/thrift_upgrade_test.py:28:1: E302 expected 2 blank lines, 
> found 1
> ./upgrade_tests/thrift_upgrade_test.py:37:1: E302 expected 2 blank lines, 
> found 1
> {noformat}
> Running {{pep8}} is also failed because it's renamed to {{pycodestyle}}.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to