flake8 --install-hook option doesn't work about flake8 HOT 6 CLOSED

In GitLab by @sigmavirus24 on Sep 23, 2014, 19:12

I can reproduce this. Thanks for the report @nathan12343

from flake8.

In GitLab by @sigmavirus24 on Sep 23, 2014, 20:33

So this is being triggered in pep8. I'm curious why we're only seeing this now.

I disabled multiprocessing to ensure it wasn't that. If there's a config file in the directory, it'll work just fine. I wonder when exactly this broke.

from flake8.

In GitLab by @jcwilson on Oct 6, 2014, 13:01

This is actually due to a larger issue of pep8 doing it's own option parsing which is being elided by flake8. For instance, [flake8] exclude patterns are not processed correctly for the internal pep8 delegation. flake8 sees it as a simple comma-delimited string while pep8 processes it into a list of exclude patterns. Since pep8 iterates over the exclude value when testing the patterns, it ends up comparing package and module names against the individual characters in the string passed from flake8, not the tokenized items that it expects to receive.

Should I open a separate issue for this?

from flake8.

In GitLab by @sigmavirus24 on Oct 6, 2014, 13:42

@jcwilson can you rewrite that perhaps to be more coherent? As best I can tell your comment does not relate to this issue at all.

from flake8.

In GitLab by @jcwilson on Oct 6, 2014, 14:32

Sure, let me create another issue so as not to add more noise to this one. Sorry for the confusing write up.

from flake8.

In GitLab by @sigmavirus24 on Oct 9, 2014, 14:47

Thanks for reporting this @nathan12343. It should be fixed as of 7fd24c2 and the soon-to-be-released 2.2.4. Cheers!

from flake8.