Skip to content

Use Into<String> in new type constructors - #169

Draft
gibbz00 wants to merge 3 commits into
ramosbugs:mainfrom
gibbz00:into_string
Draft

gibbz00 wants to merge 3 commits into
ramosbugs:mainfrom
gibbz00:into_string

Conversation

@gibbz00

@gibbz00 gibbz00 commented May 11, 2024

Copy link
Copy Markdown

Fixes: #168

Relies on: ramosbugs/oauth2-rs#275

f95b00f needs to be removed once the oauth2 PR released. Marking PR as draft until it's done.

✌️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use Into<String> or ToString::to_string rather than requiring owned Strings up front.

1 participant