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

Pylint crash with StopIteration when used with --limit-inference-results 0 #4631

Closed
ghost opened this issue Jun 30, 2021 · 3 comments · Fixed by #4632 or pylint-dev/astroid#1081
Closed

Pylint crash with StopIteration when used with --limit-inference-results 0 #4631

ghost opened this issue Jun 30, 2021 · 3 comments · Fixed by #4632 or pylint-dev/astroid#1081
Assignees
@Pierre-Sassoulas
Labels
Blocker 🙅 Blocks the next release Crash 💥 A bug that makes pylint crash Needs astroid update Needs an astroid update (probably a release too) before being mergable
Milestone
2.9.1

Comments

@ghost
Copy link

ghost commented Jun 30, 2021

Steps to reproduce

  1.  pylint --fail-under 9 --limit-inference-results 0 ./spotdl

    on any file/files that import from modules that haven't been installed

  2. Discovered on GitHub actions run that you can check here.

pylint --version output

Result of pylint --version output:

2.9.0

Notes

  1. Sorry for not strictly adhering to the template. All the files, previous checks and current check outputs can be seen on GitHub itself.

  2. This is an issue I found only with 2.9.0
@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.9.1 milestone Jun 30, 2021
@Pierre-Sassoulas Pierre-Sassoulas added Blocker 🙅 Blocks the next release Crash 💥 A bug that makes pylint crash labels Jun 30, 2021
@Pierre-Sassoulas Pierre-Sassoulas mentioned this issue Jun 30, 2021
Merged
4 tasks
@Pierre-Sassoulas Pierre-Sassoulas changed the title Pylint errors out (with lengthy traceback) on import-error Pylint crash with StopIteration when used with --limit-inference-results 0 Jun 30, 2021
@Pierre-Sassoulas
Copy link
Member

Thanks for opening the issue, I can reproduce this problem.

Pierre-Sassoulas added a commit to Pierre-Sassoulas/pylint that referenced this issue Jun 30, 2021
@Pierre-Sassoulas
Add a regression test for issue pylint-dev#4631
2bbc41a
@Pierre-Sassoulas Pierre-Sassoulas self-assigned this Jun 30, 2021
@Pierre-Sassoulas Pierre-Sassoulas mentioned this issue Jun 30, 2021
Merged
Pierre-Sassoulas added a commit to Pierre-Sassoulas/pylint that referenced this issue Jun 30, 2021
@Pierre-Sassoulas
Add a regression test for issue pylint-dev#4631
f1d19b7
@Pierre-Sassoulas Pierre-Sassoulas added the Needs astroid update Needs an astroid update (probably a release too) before being mergable label Jun 30, 2021
Pierre-Sassoulas added a commit to Pierre-Sassoulas/pylint that referenced this issue Jun 30, 2021
@Pierre-Sassoulas
Add a regression test for issue pylint-dev#4631
4f285ff
@Pierre-Sassoulas Pierre-Sassoulas mentioned this issue Jun 30, 2021
Merged
Pierre-Sassoulas added a commit to Pierre-Sassoulas/pylint that referenced this issue Jun 30, 2021
@Pierre-Sassoulas
Add a regression test for issue pylint-dev#4631
ccd01ea
@Pierre-Sassoulas
Copy link
Member

This should be fixed if upgrading to astroid 2.6.2 manually, and will be fixed in pylint 2.9.1.

Pierre-Sassoulas added a commit that referenced this issue Jun 30, 2021
@Pierre-Sassoulas
Add a regression test for issue #4631
5fcc6f5
@ghost
Copy link
Author

ghost commented Jun 30, 2021

Thanks for the timely help.

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

@Pierre-Sassoulas Pierre-Sassoulas

Labels
Blocker 🙅 Blocks the next release Crash 💥 A bug that makes pylint crash Needs astroid update Needs an astroid update (probably a release too) before being mergable
Projects
None yet
Milestone
2.9.1
1 participant
@Pierre-Sassoulas