search

serenity-rs / poise

Discord bot command framework for serenity, with advanced features like edit tracking and flexible argument parsing
MIT License
644 stars 114 forks source link

CreateReply on `next` should use `impl Into<Cow<'a, str>>` instead of `impl Into<String>` #284

Open jamesbt365 opened 3 months ago

jamesbt365 commented 3 months ago

serenity@next does this on most builders and there doesn't seem to be a reason to have a full owned String other than funky lifetimes?

Maybe I'm missing something.