Skip to content

Conversation

JohnTitor
Copy link
Member

Fixes #4991

This approach may be fragile though...

changelog: Add suggestions for if_let_some_result

@JohnTitor
Copy link
Member Author

JohnTitor commented Jan 10, 2020

Also, renamed files and module to if_let_some_result, I think it's more appropriate name.

@bors
Copy link
Contributor

bors commented Jan 11, 2020

☔ The latest upstream changes (presumably #5040) made this pull request unmergeable. Please resolve the merge conflicts.

@JohnTitor JohnTitor force-pushed the add-sugg-some-result branch from f4a06b9 to 5ded8ef Compare January 11, 2020 21:47
@bors
Copy link
Contributor

bors commented Jan 13, 2020

☔ The latest upstream changes (presumably #5042) made this pull request unmergeable. Please resolve the merge conflicts.

@JohnTitor JohnTitor force-pushed the add-sugg-some-result branch from 5ded8ef to 56feda1 Compare January 13, 2020 14:09
@JohnTitor JohnTitor added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jan 13, 2020
Copy link
Member

@flip1995 flip1995 left a comment

Choose a reason for hiding this comment

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

File/module rename is fin 👍

@JohnTitor JohnTitor force-pushed the add-sugg-some-result branch from 56feda1 to 7e76a19 Compare January 19, 2020 01:14
@flip1995
Copy link
Member

Thanks, now this should be a rock solid suggestion!

@bors r+

@bors
Copy link
Contributor

bors commented Jan 19, 2020

📌 Commit c9f8d03 has been approved by flip1995

bors added a commit that referenced this pull request Jan 19, 2020
Add suggestions for `if_let_some_result`

Fixes #4991

This approach may be fragile though...

changelog: Add suggestions for `if_let_some_result`
@bors
Copy link
Contributor

bors commented Jan 19, 2020

⌛ Testing commit c9f8d03 with merge fd0428f...

@bors
Copy link
Contributor

bors commented Jan 19, 2020

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing fd0428f to master...

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.

Add suggestion to if_let_some_result lint
4 participants