-
-
Notifications
You must be signed in to change notification settings - Fork 265
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
Don't calculate slots when MRO parsing fails. #1089
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Probably the astroid fix we need for pylint-dev/pylint#4688, thank you ! |
These two issues are orthogonal as per my understanding. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know how I would test that, but would it be possible to add an automated test for this change ?
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank for adding a test, I'm going to wait for another reviewer before merging :)
Steps
Description
slots()
won't now fail if it can't parse the mro object.I'm not sure that's the right way to fix that.
Type of Changes
Related Issue
#1088
Thanks!