search

hayd / pep8radius

PEP8 clean only the parts of the files touched since the last commit, a previous commit or (the merge-base of) a branch.
MIT License
183 stars 9 forks source link

Drop autopep8 #84

Open hayd opened 7 years ago

hayd commented 7 years ago

Just use yapf. At some point critical parts of autopep8 were removed (due to IMO a minor edge case/bug). It's probably easiest to either

  1. drop autopep8 completely.
  2. make yapf the default (and set the version of autopep8 to when it worked)
  3. make yapf the default and vendorize the earlier version / extract the relevant removed code.

I'm loathe to do 3, and 1. is easiest.

Personally, I've been using yapf exclusively for some time.