Skip to content

Conversation

@dswij
Copy link
Member

@dswij dswij commented Jan 27, 2022

closes #8345

changelog: [map_flatten] Add docs info for Option case

@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 Jan 27, 2022
Copy link
Contributor

@giraffate giraffate left a comment

Choose a reason for hiding this comment

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

LGTM with nits

@giraffate
Copy link
Contributor

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Jan 28, 2022

📌 Commit 515ed80 has been approved by giraffate

@bors
Copy link
Contributor

bors commented Jan 28, 2022

⌛ Testing commit 515ed80 with merge fb94992...

@bors
Copy link
Contributor

bors commented Jan 28, 2022

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

@bors bors merged commit fb94992 into rust-lang:master Jan 28, 2022
@dswij dswij deleted the 8345 branch January 28, 2022 05:40
@ParkMyCar
Copy link
Contributor

Thanks @dswij!

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.

map_flatten lint docs should include Option cases

5 participants