autopep8 occasionally gives invalid syntax for E501 (80+ character lines) errors

[hhatto]

autopep8 occasionally gives invalid syntax for E501 (80+ character lines) errors

[dellis23]

Odd. My version shows I have 0.6.3 installed, but I just tried cloning from github and doing the following:

python autopep8/autopep8.py over_80_char.py

Now the output gives the following valid:

mydict['longkey']['key2'][
    'longkey3'] = 'my long string goes here that goes well past line 80'

My other issue of ignores not working seems to be solved when cloning and calling directly as well, so it seems that there is some sort of dichotomy here that has either been solved between the official release and github, or some difference between the command line version and when calling directly.

[dellis23]

Okay, trying to narrow this down further, I ran the commands from the autopep8.py file in my site-packages directory and am still getting the same issue.

[dellis23]

This appears to simply be an issue with the version in pip being broken versus the current state of the code in the repo. I'm not really familiar with how updating packages in pip works, as well as who should close an issue and when, so I will leave this open.

If anyone could enlighten me on how these two processes generally function I'd appreciate it. I'd like to contribute to open source more, and I think it would be helpful to know how these things are best done.

[myint]

I've confirm your findings about 0.6.3.

I made some stylistic improvements to the line shortening algorithm. But I don't recall fixing anything. I'll compare with 0.6.3 to see what exactly changed.

[myint]

It looks like I did fix it. The code previously didn't fully check for syntax problems before breaking the line. We need to upgrade the version on PyPi.

@hhatto, could you bump the version? Thanks.

[hhatto]

version 0.6.4 has been released.