Async function remove_cog not awaited

[shtlrs]

If you take a look at this line

bot/bot/exts/moderation/watchchannels/_watchchannel.py

Line 131 in 71fbd5a

self.bot.remove_cog(self.__class__.__name__)

You'll see that the remove_cog async func isn't awaited.

[shtlrs]

Can someone assign this to me? I'll include it in the PR for the other bug ino working on.

[mbaruh]

Looks like a remnant of the move to discord.py 2.0.

I'm not sure awaiting it works. iirc raising an error will cause it to unload. Worth checking.

[shtlrs]

@mbaruh do you mean the fact that it isn't awaited will throw and thus would still unload the cog?

I noticed it upon unloading a cog actually, and I thought it was ugly to have that exception in our logs.

BTW, couldn't this have been avoided if we used something like mypy? Why don't we actually have that in our project?