-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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 skip_while_next
lint
#5067
Add skip_while_next
lint
#5067
Conversation
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.
LGTM overall.
@bors delegate+ r=me if you want to split up the messages of other lints in a separate PR, instead of in this PR. |
✌️ @JohnTitor can now approve this pull request |
Ah sorry, "later" was a little vague. I intended to follow up on another PR. So let's |
📌 Commit bec5b69 has been approved by |
Add `skip_while_next` lint Fixes #4036 changelog: Add `skip_while_next` lint
☀️ Test successful - checks-travis, status-appveyor |
Fixes #4036
changelog: Add
skip_while_next
lint