Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Semicolon is now always removed #338

Closed
fpuga opened this issue Jul 14, 2017 · 4 comments
Closed

Semicolon is now always removed #338

fpuga opened this issue Jul 14, 2017 · 4 comments

Comments

@fpuga
Copy link

fpuga commented Jul 14, 2017

In files like the next one the semicolons are not removed, but pycodestyle raises a E703 error.

# foo.py
MY_CONST = [
    {'A': 1},
    {'B': 2}
];

System Info:

autopep8 1.3.2 (pycodestyle: 2.3.1)
Python 2.7.9
Linux Rojo 4.4.0-83-generic #106-Ubuntu SMP Mon Jun 26 17:54:43 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

I upgrade autopep8 to the available version on github and i'm launching it like this:

autopep8 --in-place --recursive --select E,W --aggressive --aggressive --experimental --max-line-length=119 foo.py
@fpuga
Copy link
Author

fpuga commented Jul 14, 2017

I think that the error is happening when checking if it's a compount stmt

@myint
Copy link
Collaborator

myint commented Jul 15, 2017

I can confirm. This is due to the workaround in ca02ec6.

hhatto pushed a commit that referenced this issue Oct 10, 2017
@hhatto hhatto mentioned this issue Oct 10, 2017
@hhatto
Copy link
Owner

hhatto commented Oct 13, 2017

@fpuga
autopep8 version 1.3.3 has been released.
Can you try it as it has been fixed in the latest version?

@hhatto hhatto closed this as completed Oct 24, 2017
@fpuga
Copy link
Author

fpuga commented Feb 9, 2018

@hhatto it works like a charm. Sorry for the late answer I didn't review my notifications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants