Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

PEP257 not being run #72

Closed
dan-c-underwood opened this issue Aug 25, 2015 · 3 comments
Closed

PEP257 not being run #72

dan-c-underwood opened this issue Aug 25, 2015 · 3 comments
Labels

Comments

@dan-c-underwood
Copy link

No PEP257 errors are being reported despite flake8-pep257 or flake8-docstrings installed (tested with both).

Running flake8 in the command line on an example Python file gives the output:

build.py:1:1: D100 Missing docstring in public module
build.py:8:1: D103 Missing docstring in public function
build.py:21:1: D103 Missing docstring in public function

However these are not displayed by this linter. In the package settings I've tried changing it to recognise E, W, D rather than just E, W, but this has no effect.

@Arcanemagus
Copy link
Member

You know, I was wondering why I wasn't seeing any errors related to that...

Turns out this is a bug in the regex that filters out the input (because flake8 can't provide nice json output...). This happens after the output of the program, hence why the setting has no effect. I'll throw up a PR that fixes this shortly.

@cemsbr
Copy link
Contributor

cemsbr commented Nov 11, 2015

I'm having this issue.

@Arcanemagus
Copy link
Member

As it looks like you already discovered, this isn't actually the issue you are experiencing @cemsbr.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants