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

Stefania commented on CASSANDRA-10066:
--------------------------------------

Python 2.7.6, pep8 installed via pip:

{code}
stefi@lila:~/git/cstar/cassandra$ python --version
Python 2.7.6
stefi@lila:~/git/cstar/cassandra$ which python
/usr/bin/python
stefi@lila:~/git/cstar/cassandra$ ll /usr/bin/python
lrwxrwxrwx 1 root root 24 Sep  5 19:14 /usr/bin/python -> 
/etc/alternatives/python*
stefi@lila:~/git/cstar/cassandra$ ll /etc/alternatives/python 
lrwxrwxrwx 1 root root 18 Sep  5 19:45 /etc/alternatives/python -> 
/usr/bin/python2.7*

stefi@lila:~/git/cstar/cassandra$ pep8 --version
1.6.2
stefi@lila:~/git/cstar/cassandra$ which pep8
/usr/local/bin/pep8
stefi@lila:~/git/cstar/cassandra$ ll /usr/local/bin/pep8 
-rwxr-xr-x 1 root root 207 Sep  7 14:44 /usr/local/bin/pep8*
{code}

If you are sure the version of pep8 that runs on the CI machine is fine then 
I'd say the patch is +1.



> Bring cqlsh into PEP8 compliance
> --------------------------------
>
>                 Key: CASSANDRA-10066
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10066
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Philip Thompson
>            Assignee: Philip Thompson
>            Priority: Minor
>              Labels: cqlsh
>             Fix For: 3.x, 2.1.x, 2.2.x
>
>
> In order for us to begin running flake8 against cqlsh.py in CI, it would be 
> helpful if it were already PEP8 complaint, with the exception of using 120 
> character lines.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to