Current coverage is 47.30% (diff: 50.00%)

Merging #414 into master will increase coverage by <.01%

@@             master       #414   diff @@
==========================================
  Files            56         56          
  Lines          4833       4839     +6   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           2286       2289     +3   
- Misses         2547       2550     +3   
  Partials          0          0          

Powered by Codecov. Last update 8d73f62...553e042

Thanks! If you are going to do more of these PEP8 fixes can you roll them up into one PR?

@beardypig are you gonna accept all of them? Because some of them was not merged in other projects. I put all non-aggressive (default) autopep8 fixes in this PR.

Most of them I will accept, if you put them in separate commits with the appropriate message it will be easier to filter out the ones that we don't want.
Can you drop the line length limit (E501 I think).

All of those look reasonable to me, thanks :)

I'm done. Except E265, don't know about this one.

Cool, looks good... E309 isn't part of PEP8, it's part of PEP257 and it doesn't look like autopep8 does it correctly.

@beardypig Ok, reverted it.

@beardypig these look good to me, any other concerns/questions? If not @stepshal can you rebase after #417 is merged?

@gravyboat No. Done.

@beardypig If you don't have any additional comments when you're around tomorrow let's squash and merge this.

Looking good to merge. @stepshal thanks for breaking them out in to separate commits, that must have been a bit of a pain :) Good job!

@beardypig Do you want to keep all these PRs separate in the event we need to roll back? If not I would like to squash since it's a large number of commits and it might crowd the history.

Yeah squash it!

Done, thanks @stepshal!