Run flake8

[bcipolli]

Great to have a template that follows PEP8 and passes pyflakes, even if most projects themselves choose not to follow the cleanliness of such a sparkling template :)

[jesford]

flake8 is awesome!

However, I was not able to run it using the command provided in the README:

flake8 --ignore N802,N806 `find . -name *.py | grep -v setup.py | grep -v /doc/`

and also not when removing the (unintentional?) extra backticks:

flake8 --ignore N802,N806 find . -name *.py | grep -v setup.py | grep -v /doc/

It gives me this error:

Usage: flake8 [options] input ...

flake8: error: no such option: -n

But it runs fine if I just do this:

flake8 --ignore N802,N806 *.py | grep -v setup.py | grep -v /doc/

Not sure if its a typo in the README (I'm happy to submit a PR if so) or if its just me...

Thanks for this resource @arokem! Super helpful.

[arokem]

Just a guess here: should that be a -- before the name (two dashes
instead of one)?

On Wed, Feb 10, 2016 at 7:06 PM, Jes Ford [email protected] wrote:

flake8 is awesome!

However, I was not able to run it using the command provided in the README:

flake8 --ignore N802,N806 find . -name *.py | grep -v setup.py | grep -v /doc/

and also not when removing the (unintentional?) extra backticks:

flake8 --ignore N802,N806 find . -name *.py | grep -v setup.py | grep -v /doc/

It gives me this error:

Usage: flake8 [options] input ...

flake8: error: no such option: -n

But it runs fine if I just do this:

flake8 --ignore N802,N806 *.py | grep -v setup.py | grep -v /doc/

Not sure if its a typo in the README (I'm happy to submit a PR if so) or
if its just me...

Thanks for this resource @arokem https://github.com/arokem! Super
helpful.

—
Reply to this email directly or view it on GitHub
#11 (comment).

[jesford]

No, that gives me the same error. Does it work for you?

[arokem]

Sorry - was really just guessing.

This does work for me (but might not work inside the Travis setup -- I
think that's what the README refers to?)

flake8 --ignore N802,N806 shablona/*.py

(issued from the top level repo directory).

On Wed, Feb 10, 2016 at 7:14 PM, Jes Ford [email protected] wrote:

No, that gives me the same error. Does it work for you?

—
Reply to this email directly or view it on GitHub
#11 (comment).

[rbiswas4]

Hi Jes,

Escaping the * might work:

find . -name _.py should probably be
find . -name _.py

The backticks are meant to run the find command and locate the *.py within
directories and then run flake 8 on that list. So, I think you could simply
run the flake8 command on an explicit file list to see if it is the flake8
command that is a problem or building the list of files.

Thanks,
Rahul

On Wed, Feb 10, 2016 at 7:22 PM, Ariel Rokem [email protected]
wrote:

Sorry - was really just guessing.

This does work for me (but might not work inside the Travis setup -- I
think that's what the README refers to?)

flake8 --ignore N802,N806 shablona/*.py

(issued from the top level repo directory).

On Wed, Feb 10, 2016 at 7:14 PM, Jes Ford [email protected]
wrote:

No, that gives me the same error. Does it work for you?

—
Reply to this email directly or view it on GitHub
<#11 (comment)
.

—
Reply to this email directly or view it on GitHub
#11 (comment).

[jesford]

Thanks @rbiswas4 that works perfect!