Zephyrkul

Results 16 comments of Zephyrkul

I'll also note that commands themselves still accept `help=` as expected ![Minimal example](https://user-images.githubusercontent.com/23347632/187094216-f887af0d-f022-4a8f-b63a-43b26f8dec76.png)

Getting this traceback in my console whenever I get a CommandNotFound when I try this PR out: ``` Ignoring exception in on_command_error Traceback (most recent call last): File "C:\Users\...\Desktop\Github\Red-DiscordBot\.venv\lib\site-packages\discord\client.py", line...

It looks like `@bot.hybrid_command` and `@bot.hybrid_group` haven't been implemented here yet to return these new classes, is this planned for this PR?

fwiw `avatar.url` is perfectly valid under discord.py 2.0 - you're thinking of `avatar_url`. The `Embed.Empty` fix is necessary, though.

Would it be possible it to make these payload classes into [NamedTuple](https://docs.python.org/3/library/typing.html#typing.NamedTuple)s instead? That should be a free performance improvement since these classes don't need to be modified after they're...

`black` is failing on this PR but this is due to files in other areas of the repo (`ccrole/ccrole.py` and `fifo/task.py`, specifically) - the only file this PR touches would...

Ah, sorry, no it doesn't. I didn't see that comment further down. Worth noting that CC doesn't handle default arguments either.

May as well just split the difference and have a confirmation menu before downgrading the ban to a tempban.

> - Prompt will be skipped if `ctx.assume_yes` evaluates to `True` I don't think allowing tempban to be run on a schedule would be best here.