Skip to content
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

bugfix: fix Pylint collector (broken since Pylint 1.6.0) #193

Merged
merged 1 commit into from
May 2, 2017

Conversation

arthepsy
Copy link
Contributor

Since pylint 1.6.0, no messages are reporter via prospector. See for example #188 and #183.

It's because of this commit - pylint-dev/pylint@dc07196 - which removes handle_message body in BaseReporter.

This pull requests fixes the issue.

@coveralls
Copy link

coveralls commented Oct 20, 2016

Coverage Status

Coverage decreased (-0.04%) to 59.13% when pulling ca97868 on arthepsy:bugfix-pylint into a5606a9 on landscapeio:master.

@jayktaylor
Copy link

jayktaylor commented Apr 19, 2017

Is this going to be merged anytime soon? It's much better than having to downgrade to 1.5.6, and the original issue relating to this PR #183 has been open since last August. Of course, using pylint 1.7 could fix this altogether (assuming, although maybe it wouldn't), but there's an issue with that too (#206).

@carlio carlio merged commit ca97868 into prospector-dev:master May 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants