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

Remove the pylint locally-enabled message suppression #309

Closed
wants to merge 1 commit into from

Conversation

pgjones
Copy link
Contributor

@pgjones pgjones commented Dec 2, 2018

As of pylint 2.2 this message is no longer present in pylint, and
hence does not need suppressing. Sadly however this line errors with pylint>= 2.2 and hence needs removing.

pgjones added a commit to pgjones/wsproto that referenced this pull request Dec 2, 2018
This is required till prospector is fixed, See
landscapeio/prospector#309.
hence does not need suppressing. Sadly however this line errors with
pylint>=2.2 and hence needs removing.
@coveralls
Copy link

coveralls commented Dec 2, 2018

Coverage Status

Coverage decreased (-0.01%) to 77.688% when pulling 0276159 on pgjones:master into 8bb1970 on PyCQA:master.

pgjones added a commit to pgjones/wsproto that referenced this pull request Dec 2, 2018
This is required till prospector is fixed, See
landscapeio/prospector#309.
pgjones added a commit to pgjones/wsproto that referenced this pull request Dec 4, 2018
This is required till prospector is fixed, See
landscapeio/prospector#309.
pgjones added a commit to pgjones/wsproto that referenced this pull request Dec 5, 2018
This is required till prospector is fixed, See
landscapeio/prospector#309.
pgjones added a commit to pgjones/wsproto that referenced this pull request Dec 5, 2018
This is required till prospector is fixed, See
landscapeio/prospector#309.
pgjones added a commit to pgjones/wsproto that referenced this pull request Dec 5, 2018
This is required till prospector is fixed, See
landscapeio/prospector#309.
pgjones added a commit to python-hyper/wsproto that referenced this pull request Dec 22, 2018
This is required till prospector is fixed, See
landscapeio/prospector#309.
@chocoelho
Copy link
Contributor

@pgjones this commit was cherry-picked in another PR, thanks for the contribution

@chocoelho chocoelho closed this Jun 12, 2019
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.

None yet

3 participants