[Suggestion] CNext: Respond with string for Task<string> Commands

Summary

Just a suggestion: Make Task<string> respond with the string returned. I don’t want to touch cnext and get sniped by emzi for breaking that (it’s Emziware)

Details

[Command("reply")]
public async Task<string> ReplyAsync(CommandContext ctx)
{
    return "Hello World!";
}

and the bot should respond with Hello World!

Notes

I don’t want to get killed by Emzi for touching cnext or adding stuff to CNext without discussing

[Emzi0767]

This is one of the things I considered and rejected early on. Adds unnecessary complexity. If anything, I’d use the return types to perhaps indicate a condition other than success, but even then, added complexity for no gain at all.

[Emzi0767]

There’s also the performance aspect. This would cause a significant performance hit to command registration code, possibly command invocation code, and with coming changes to CNext, it’d make the whole CNext codebase extremely hard to debug, test, and maintain.