ironman5366 / W.I.L.L

A python written personal assistant
https://willbeddow.com
MIT License
377 stars 58 forks source link

PEP8 #4

Closed brenttaylor closed 8 years ago

brenttaylor commented 8 years ago

The code base doesn't adhere to a good number of the PEP8 standards and it really should. It gives everyone, especially code contributors, a style guide they can adhere too. Especially if they are using a linter (and it looks like you're using IDEA, so you absolutely have a PEP8 linter).

A few examples of issues:

There's actually a lot more, but it's all pretty easily changed. I can work on it before #3 if you'd like.

ironman5366 commented 8 years ago

Sure, go write ahead. I used idea for part of it, but during development I've probably used 20 different editors and and 4 different os's.

narzeja commented 8 years ago

I made a PR (#7). Take a look and see if I covered everything PEP8-wise.

ironman5366 commented 8 years ago

Amazing, thanks so much