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

Add regression tests for issue #3711 #4777

Conversation

Pierre-Sassoulas
Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas commented Jul 30, 2021

Type of Changes

Type
βœ“ πŸ› Bug fix

Description

This is a regression tests for pylint-dev/astroid#1111

Closes #3711

@Pierre-Sassoulas Pierre-Sassoulas added Bug πŸͺ² Control flow Requires control flow understanding labels Jul 30, 2021
@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.10.0 milestone Jul 30, 2021
@Pierre-Sassoulas Pierre-Sassoulas added the Needs astroid update Needs an astroid update (probably a release too) before being mergable label Jul 30, 2021
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the regression-tests-for-astroid-1111 branch 4 times, most recently from 4f5b0f9 to a9749a1 Compare August 3, 2021 20:46
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the regression-tests-for-astroid-1111 branch from 120da70 to a55157a Compare August 4, 2021 07:33
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the regression-tests-for-astroid-1111 branch from a55157a to 1774a28 Compare August 4, 2021 18:22
@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.587% when pulling 1774a28 on Pierre-Sassoulas:regression-tests-for-astroid-1111 into fbdab02 on PyCQA:main.

@Pierre-Sassoulas Pierre-Sassoulas merged commit 02e5289 into pylint-dev:main Aug 4, 2021
@Pierre-Sassoulas Pierre-Sassoulas deleted the regression-tests-for-astroid-1111 branch August 4, 2021 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug πŸͺ² Control flow Requires control flow understanding Needs astroid update Needs an astroid update (probably a release too) before being mergable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

False positive undefined-loop-variable with pylint 2.5.3
2 participants