Skip to content

[useless_conversion]: pluralize if there are multiple .into_iter() calls#10881

Merged
bors merged 1 commit into
rust-lang:masterfrom
y21:explicit-into-iter-fn-arg-followup
Jun 3, 2023
Merged

[useless_conversion]: pluralize if there are multiple .into_iter() calls#10881
bors merged 1 commit into
rust-lang:masterfrom
y21:explicit-into-iter-fn-arg-followup

Conversation

@y21

@y21 y21 commented Jun 3, 2023

Copy link
Copy Markdown
Member

context: #10814 (comment)

changelog: [useless_conversion]: pluralize if there are multiple .into_iter() calls in the chain

r? @llogiq

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jun 3, 2023

@llogiq llogiq left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, that was quick. One small nit, otherwise r=me.

Comment thread clippy_lints/src/useless_conversion.rs Outdated
@y21 y21 force-pushed the explicit-into-iter-fn-arg-followup branch from ba6a9d8 to 5a7e33e Compare June 3, 2023 17:39
@llogiq

llogiq commented Jun 3, 2023

Copy link
Copy Markdown
Contributor

Thank you!

@bors r+

@bors

bors commented Jun 3, 2023

Copy link
Copy Markdown
Contributor

📌 Commit 5a7e33e has been approved by llogiq

It is now in the queue for this repository.

@bors

bors commented Jun 3, 2023

Copy link
Copy Markdown
Contributor

⌛ Testing commit 5a7e33e with merge fdb0b04...

@bors

bors commented Jun 3, 2023

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing fdb0b04 to master...

@bors bors merged commit fdb0b04 into rust-lang:master Jun 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants