Feat/help command

[J3y0]

This PR solves #15 issue.

Problem was the bot didn't have the guild intent which is necessary for the help command (didn't find it on the internet so I did find it by trying repeatedly with different intents...).

This PR brings:

• A refactor of craft_intent function where intents accorded to the bot are clearly stated (it was impossible for me to understand what intents were linked to the value passed in argument to the Intent() constructor.
• A custom HelpCommand which implements the class HelpCommand of the discord library. It is very simple for now and has to be considered as a base for future and more complex implementation.
• As the HelpCommand implementation is directly given to the Bot() constructor, the help command was not in the COG and couldn't be subject to logging with the before_invoke decorator. This is why, I added logging into the on_command event. Then, logging is automatically done when a command is triggered

[ctmbl]

Seems great! Just a few question but you did a great job actually!!

Thanks for rewriting the craft_intents fucntion it was a mess ahah
and the commands desc looks great too!

[J3y0]

Thanks for your review ! Feel free to ask some features you would like to see in the custom help command ! I add the strict minimum right now !

[J3y0]

I couldn't get a perfect pylint as it misses the fact I am implementing correctly the methods of the HelpCommand class. I looked at the source code to be sure.

This is why, I tried to look at a similar issue and it turns out there is an issue on the arguments-differ feature of pylint.

[ctmbl]

It was a long time ago so I may have missed some caveats but seems ready to me!

[ctmbl]

@J3y0

I couldn't get a perfect pylint as it misses the fact I am implementing correctly the methods of the HelpCommand class. I looked at the source code to be sure.

Problem is that you won't be able to merge the PR if pylint doesn't pass...
I try to avoid doing this but you can disable this pylint feature (globally in the pylintrc or locally in the source code) if you need 😕

This is why, I tried to look at a similar issue and it turns out there is an pylint-dev/pylint#1553 on the arguments-differ feature of pylint.

hmm the issue is closed don't know if it should be solved or not to be honest

[ctmbl]

once @J3y0 will have resolved/answered the last thread we'll be able to merge this!