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.

Sign up for GitHub

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

Jump to bottom

pytype not compatible with pylint 2.4.0+, astroid 2.3.0+ #423

Closed
rchen152 opened this issue Sep 27, 2019 · 1 comment · Fixed by #476
Closed

pytype not compatible with pylint 2.4.0+, astroid 2.3.0+ #423

rchen152 opened this issue Sep 27, 2019 · 1 comment · Fixed by #476
Assignees
@rchen152
Labels
bug

Comments

@rchen152
Copy link
Contributor

With the September 24 release of pylint, it has started reporting many lint errors in pytype that it wasn't previously. Downgrading to pylint 2.3.1 causes the errors to go away.

With the September 24 release of astroid, pylint has started crashing with a recursion error at the end of analysis. Downgrading to astroid 2.2.5 causes the crash to go away. The recursion error looks like pylint-dev/pylint#3123.
@rchen152 rchen152 added the bug label Sep 27, 2019
@rchen152
Copy link
Contributor Author

Example of failures: https://travis-ci.org/google/pytype/builds/590131868

rchen152 added a commit that referenced this issue Sep 27, 2019
@rchen152
Temporarily pin to an older version of pylint.
087c9e3 
See #423.
@rchen152 rchen152 mentioned this issue Sep 27, 2019
Merged
@rchen152 rchen152 self-assigned this Dec 11, 2019
rchen152 added a commit that referenced this issue Dec 12, 2019
@rchen152
Remove an unnecessary list comprehension.
27d1efb 
The unnecessary comprehension was flagged by the most recent upstream release
of pylint.

For #423.

PiperOrigin-RevId: 285068776
@rchen152 rchen152 mentioned this issue Dec 12, 2019
Merged
rchen152 added a commit that referenced this issue Dec 12, 2019
@rchen152
Make pytype work with the most recent version of pylint.
7ce1fa1 
Fixes #423.
@rchen152 rchen152 mentioned this issue Dec 12, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rchen152 rchen152

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make pytype work with the most recent version of pylint. google/pytype
1 participant
@rchen152