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

Suggest unwrap_or_default when closure returns "".to_string #9421

Merged
merged 1 commit into from
Sep 6, 2022

Conversation

xphoniex
Copy link
Contributor

@xphoniex xphoniex commented Sep 3, 2022

Closes #9420

changelog: [unwrap_or_else_default]: suggest unwrap_or_default() instead of unwrap_or_else with a closure that returns an empty to_string.

@rust-highfive
Copy link

r? @giraffate

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 3, 2022
Signed-off-by: xphoniex <xphoniex@users.noreply.github.com>
@giraffate
Copy link
Contributor

@bors r+

Thanks!

@bors
Copy link
Collaborator

bors commented Sep 6, 2022

📌 Commit b21d9d3 has been approved by giraffate

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Sep 6, 2022

⌛ Testing commit b21d9d3 with merge 7babd1b...

@bors
Copy link
Collaborator

bors commented Sep 6, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: giraffate
Pushing 7babd1b to master...

@bors bors merged commit 7babd1b into rust-lang:master Sep 6, 2022
@xphoniex xphoniex deleted the fix-#9420 branch September 7, 2022 03:07
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.

Suggest unwrap_or_default when returning empty String in closure
4 participants