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

Return Result in try_as_owned_mut and try_into_owned. #2

Merged
merged 1 commit into from Apr 28, 2023

Conversation

kauppie
Copy link
Owner

@kauppie kauppie commented Apr 28, 2023

Closes #1

  • try_as_owned_mut and try_into_owned now return Results instead of Options with borrowed variant as the error item.
  • as_owned_mut was renamed to try_as_owned_mut.

Breaks current API and prompts for change to 0.2.0 or similar.

@kauppie kauppie added the enhancement New feature or request label Apr 28, 2023
@kauppie kauppie self-assigned this Apr 28, 2023
@kauppie kauppie merged commit d317a87 into main Apr 28, 2023
1 check passed
@kauppie kauppie deleted the 1-return-borrowed-as-an-error branch April 28, 2023 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Return Borrowed as an error
1 participant