-
Notifications
You must be signed in to change notification settings - Fork 1.8k
extend needless_collect
#14361
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
extend needless_collect
#14361
Conversation
eb133b0 to
5edcbbb
Compare
6a16dae to
2d3a3df
Compare
This comment has been minimized.
This comment has been minimized.
2d3a3df to
409b9f9
Compare
|
Ping @y21 from triage. It looks like this is waiting on review. |
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.
Not the reviewer, but this looks pretty good to me
|
r? Jarcho |
409b9f9 to
a8e509f
Compare
|
This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
Fix rust-lang#14350 Co-authored-by: Timo <[email protected]>
a8e509f to
6d7072b
Compare
|
@Jarcho Ready for review |
|
@lengyijun, for marking the PR as ready to review, please use the following command (me mentioning it will already call it, so you don't need to do anything this time): @rustbot ready Normally, rustbot will make a comment about that – weird that it didn't this time.. |
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 you.
changelog: [
needless_collect]: extend needless_collect to lint more casesFix #14350