-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add regression test for #5770 #5846
Add regression test for #5770 #5846
Conversation
Pull Request Test Coverage Report for Build 1973436590
π - Coveralls |
Ah, I remember now. The regression files aren't loaded in exactly the same way that the linter loads them. So we need to find a different way to capture the test case. |
Had to enable an extension. Better to leave it here or with the tests for this extension? |
Better to test there I think. The crash probably also occurs in the extension? |
The crash only occurs with the extension. I didn't notice because pylint's own
Sorry, just to check: you're saying move to |
Ah yeah, that's what I meant.. English π
Yes π In a separate file there. |
Type of Changes
Description
Closes #5770
Refs pylint-dev/astroid#1400