change ctx.send_help to ctx.invoke(help_command)

[ToxicKidz]

Relevant Issues

Closes #624

Description

I've changed all the ctx.send_help's to bot.get_command("help") and ctx.invoke(help_command, ctx.command.name).

Reasoning

It looks a lot better.

Screenshots

Before:

After:

Additional Details

Affected commands are .games, .snakes, .space, .emoji, .minesweeper, .movies, .status, .extensions (and its subcommands), and .bemyvalentine.

Did you:

• Join the Python Discord Community?
• If dependencies have been added or updated, run pipenv lock?
• Lint your code (pipenv run lint)?
• Set the PR to allow edits from contributors?

[Shivansh-007]

Can we add help_command a class attribute in Bot class in bot/bot.py, then we could just do:

await ctx.invoke(self.bot.help_command, ctx.command.name)

[anand2312]

Can we add help_command a class attribute in Bot class in bot/bot.py, then we could just do:

await ctx.invoke(self.bot.help_command, ctx.command.name)

bot.help_command is an attribute of commands.Bot and has to be of type HelpCommand. I don't know how overriding this with something that is not of that type will work.

[Shivansh-007]

Can we add help_command a class attribute in Bot class in bot/bot.py, then we could just do:

await ctx.invoke(self.bot.help_command, ctx.command.name)

bot.help_command is an attribute of commands.Bot and has to be of type HelpCommand. I don't know how overriding this with something that is not of that type will work.

Ah right! Forgot about the help_command, didn't meant to override, what I meant was to add another class attribute to the Bot class, so we won't need to do

help_command = self.bot.get_command("help")

everytime we are sending help on a group command.

[Akarys42]

LGTM, thanks a lot!

[ChrisLovering]

Looks good to me!