search

paetzke / py-autopep8.el

py-autopep8.el integrates autopep8 into Emacs.
GNU General Public License v3.0
79 stars 21 forks source link

Don't ignore `py-autopep8-options` when running autopep8 #9

Closed dfdeshom closed 9 years ago

dfdeshom commented 9 years ago

py-autopep8.el ignores custom options specified in py-autopep8-options. It's an issue with the way the autopep8 program runs, all options need to be specified before passing it a file. So autopep8 --select E301 --in-place file.py works, but autopep8 --in-place file.py --select E301 doesn't.

This patch appends custompy-autopep8-options options before file arguments so that they don't get ignored.

Also, I got rid of minor flycheck formatting warnings for packages.

paetzke commented 9 years ago

Thanks for your patch! I just added some tests. I will release a new version today so you can download it from MELPA.