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

manual_unwrap_or / support Result::unwrap_or #6190

Merged
merged 4 commits into from
Oct 24, 2020

Conversation

tnielens
Copy link
Contributor

Implements partially #5923.

changelog: support Result::unwrap_or in manual_unwrap_or

@rust-highfive
Copy link

r? @ebroto

(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 Oct 17, 2020
clippy_lints/src/manual_unwrap_or.rs Show resolved Hide resolved
tests/ui/manual_unwrap_or.rs Show resolved Hide resolved
tests/ui/manual_unwrap_or.rs Show resolved Hide resolved
@ebroto ebroto added S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Oct 21, 2020
clippy_lints/src/manual_unwrap_or.rs Outdated Show resolved Hide resolved
@ebroto
Copy link
Member

ebroto commented Oct 24, 2020

@bors r+

Thanks!

@bors
Copy link
Collaborator

bors commented Oct 24, 2020

📌 Commit 0d21ae0 has been approved by ebroto

@bors
Copy link
Collaborator

bors commented Oct 24, 2020

⌛ Testing commit 0d21ae0 with merge 29979ad...

@bors
Copy link
Collaborator

bors commented Oct 24, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: ebroto
Pushing 29979ad to master...

@bors bors merged commit 29979ad into rust-lang:master Oct 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants