Pep8 #333

Merged

greedo merged 3 commits from pep8 into master

2014-07-30 07:33:51 -07:00

greedo commented

2014-07-26 08:49:50 -07:00

(Migrated from github.com)

code now conforms to pep8 standards

coveralls commented

2014-07-26 08:51:32 -07:00

(Migrated from github.com)

Coverage increased (+0.82%) when pulling a0fec2f004 on greedo:pep8 into f4496ec8b2 on ryanmcgrath:master.

[![Coverage Status](https://coveralls.io/builds/1010163/badge)](https://coveralls.io/builds/1010163) Coverage increased (+0.82%) when pulling **a0fec2f004b0260dd85cd012d471e441403c4887 on greedo:pep8** into **f4496ec8b2fc9ec948d93abf035c66e43d107cb3 on ryanmcgrath:master**.

greedo commented

2014-07-29 18:24:36 -07:00

(Migrated from github.com)

@michaelhelmick

if not foo is None:

is evaluated the same way as

if foo is not None:

However the second is recommended in PEP8
See under Programming Recommendations

@michaelhelmick `if not foo is None:` is evaluated the same way as `if foo is not None:` However the second is recommended in [PEP8](http://legacy.python.org/dev/peps/pep-0008/#id41) See under `Programming Recommendations`

michaelhelmick commented

2014-07-30 07:33:43 -07:00

(Migrated from github.com)

I was unaware, thanks a lot!