Skip to content

add ResultExt::map_kind #778

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 31, 2022
Merged

add ResultExt::map_kind #778

merged 2 commits into from
May 31, 2022

Conversation

ctaggart
Copy link
Contributor

This will help with migration of errors #771.

@cataggar cataggar requested a review from rylev May 31, 2022 17:24
@cataggar cataggar changed the title add ResultExt.kind add ResultExt::kind May 31, 2022
rylev
rylev approved these changes May 31, 2022
View reviewed changes
Copy link
Contributor

@rylev rylev left a comment

Choose a reason for hiding this comment

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

Looks good though I'm not 100% sure about the name

@cataggar cataggar changed the title add ResultExt::kind add ResultExt::map_kind May 31, 2022
@cataggar cataggar merged commit 6d4d48e into Azure:main May 31, 2022
@ctaggart ctaggart deleted the kind branch June 16, 2022 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rylev rylev rylev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ctaggart @rylev @cataggar