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

fix undefined-loop-variable with NoReturn and Never #7419

Closed
wants to merge 0 commits into from
Closed

fix undefined-loop-variable with NoReturn and Never #7419

wants to merge 0 commits into from

Conversation

DetachHead
Copy link
Contributor

Type of Changes

Type
✓ 🐛 Bug fix
✓ ✨ New feature
✓ 🔨 Refactoring
✓ 📜 Docs

Description

Closes #7311

@Pierre-Sassoulas Pierre-Sassoulas added Needs backport Needs to be cherry-picked on the current patch version by a pylint's maintainer False Positive 🦟 A message is emitted but nothing is wrong with the code labels Sep 6, 2022
@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.15.2 milestone Sep 6, 2022
@Pierre-Sassoulas Pierre-Sassoulas added Needs review 🔍 Needs to be reviewed by one or multiple more persons Work in progress and removed Needs review 🔍 Needs to be reviewed by one or multiple more persons labels Sep 6, 2022
@github-actions

This comment has been minimized.

Sign in to view
@DetachHead DetachHead mentioned this pull request Sep 6, 2022
Closed
@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2022

🤖 According to the primer, this change has no effect on the checked open source code. 🤖🎉

This comment was generated for commit d48622a

@Pierre-Sassoulas Pierre-Sassoulas modified the milestones: 2.15.3, 2.15.4 Sep 16, 2022
@DanielNoord DanielNoord mentioned this pull request Sep 16, 2022
Merged
3 tasks
@Pierre-Sassoulas Pierre-Sassoulas modified the milestones: 2.15.4, 2.15.3 Sep 19, 2022
@Pierre-Sassoulas Pierre-Sassoulas removed Work in progress Needs backport Needs to be cherry-picked on the current patch version by a pylint's maintainer labels Sep 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
False Positive 🦟 A message is emitted but nothing is wrong with the code
Projects
None yet
Milestone
2.15.3
Development

Successfully merging this pull request may close these issues.

W0631:undefined-loop-variable false positive when for loop has function call that returns NoReturn
3 participants
@DetachHead @Pierre-Sassoulas @DanielNoord