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

Hotfix for pylint module run #333

Merged
merged 3 commits into from
Jun 12, 2019
Merged

Hotfix for pylint module run #333

merged 3 commits into from
Jun 12, 2019

Conversation

chocoelho
Copy link
Contributor

@chocoelho chocoelho commented Jun 12, 2019

This should fix the latest release with support for new Pylint version and Astroid releases.

Cherry picked from #309
Should resolve issue #331

c/c @carlio

chocoelho and others added 3 commits June 12, 2019 14:10
hence does not need suppressing. Sadly however this line errors with
pylint>=2.2 and hence needs removing.

(cherry picked from commit 0276159)
@chocoelho chocoelho requested a review from carlio June 12, 2019 17:22
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.8%) to 76.085% when pulling 547ae4f on hf-pylint into df3baff on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.8%) to 76.085% when pulling 547ae4f on hf-pylint into df3baff on master.

@carlio
Copy link
Member

carlio commented Jun 12, 2019

Seems to make sense to me.

@chocoelho chocoelho merged commit af00c53 into master Jun 12, 2019
@chocoelho chocoelho deleted the hf-pylint branch June 12, 2019 17:40
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

4 participants