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

Already on GitHub? Sign in to your account

Add option to output all errors immediately. #181

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants

No description provided.

Contributor

florentx commented Mar 21, 2013

I am just curious of the use case here.

Reading asynchron from a flake8 subprocess: https://github.com/schlamar/SublimeStreamingLinter

Contributor

florentx commented Mar 24, 2013

We changed this in release 1.4.2, to merge the output of the various checkers (pep8, pyflakes, etc..) and order by line number. The effect is that the output is written after each file instead of being written after each error.

If I understand correctly, your point is about performance when doing on-the-fly analysis in the editor.
I don't think this change needs to be included in pep8 core.

The wrapper could be in the plugin itself, like this:
https://gist.github.com/florentx/5232746
(it could be even simpler if flake8.run.main is changed to accept keywords and pass them to the StyleGuide constructor)

A different option would be to request the feature for Flake8, if it gives some benefit.

Owner

sigmavirus24 commented Mar 24, 2013

@schlamar do you mind taking this to the mailing list? http://mail.python.org/mailman/listinfo/code-quality

I think it's a good thing to discuss there because of the rising number of text-editors having this capability. Also, I have some ideas of my own about this but they're not relevant to pep8 and @florentx and I both agree this would be better suited in flake8 or in a plugin.

Actually, this could be done directly in a plugin without further modifications to pep8 or flake8: https://github.com/schlamar/flake8-immediate

Thanks for pointing me in the right direction!

@schlamar schlamar closed this Mar 27, 2013

@schlamar schlamar deleted the unknown repository branch Mar 27, 2013

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