New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Errors should be printed to stdout #201

Closed
gvanrossum opened this Issue Aug 28, 2016 · 8 comments

Comments

Projects
4 participants
@gvanrossum
Copy link

gvanrossum commented Aug 28, 2016

In #26 and #29 you changed errors to be printed to stderr. But actually for a linter that's quite inconvenient when you want to post-process the output. I checked flake8, pycodestyle, pyflakes and they all write their errors to stdout (and so does mypy).

Only errors that cause the program to exit immediately (e.g. command line syntax or can't open file) should go to stderr; the expected output (diagnostics about doc style) should go to stdout.

Please?

@sigmavirus24

This comment has been minimized.

Copy link
Member

sigmavirus24 commented Aug 29, 2016

@gvanrossum as a way to side-step this, is it possible for you to use flake8-docstrings?

@gvanrossum

This comment has been minimized.

Copy link

gvanrossum commented Aug 29, 2016

@Nurdok

This comment has been minimized.

Copy link
Member

Nurdok commented Aug 29, 2016

This seems reasonble, but as always, will take me some time to get to. Thanks for opening the issue.

@varunagrawal

This comment has been minimized.

Copy link
Contributor

varunagrawal commented Sep 28, 2016

I got this once we touch Oct 1!

@ghost

This comment has been minimized.

Copy link

ghost commented Oct 2, 2016

@varunagrawal Reminder to merge.

@varunagrawal

This comment has been minimized.

Copy link
Contributor

varunagrawal commented Oct 7, 2016

Pushed update to #210. Can someone review it please?

@Nurdok

This comment has been minimized.

Copy link
Member

Nurdok commented Oct 7, 2016

@varunagrawal I assure you I'll review it, but you'll have to be patient as is might take me a while.

@varunagrawal

This comment has been minimized.

Copy link
Contributor

varunagrawal commented Oct 7, 2016

@Nurdok sounds good. :)

@Nurdok Nurdok assigned Nurdok and unassigned Nurdok Oct 13, 2016

@Nurdok Nurdok closed this Dec 3, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment