Milan Smeets
Milan Smeets
That should be all (for now), fingers crossed ^^
Alright, merging and updating with new params should be done, but I haven't had time to test yet.
> I do like the changes this pr makes, but I am curious about keeping **kwargs and an argument, allow_extraneous_arguments which if False, (the default) raises TypeError about unexpected arguments....
Explicit kwargs for `CommonBotBase` and the requested `allow_extraneous_arguments` arg are still to-do; that should be all.
> Not sure if I understand the purpose of this 🤔 If you were to update, you'd need to modify your code regardless, either fixing the extra args, or adding...
There already is some prior ordering into different categories (see [this discussion](https://github.com/DisnakeDev/disnake/pull/371#discussion_r930236035)); whether that's better or worse than alphabetizing them, I am not really sure.
The documentation on this definitely needs work. As for your specific use-case, it would probably be best to have separate cogs for the production and development sides of your bot....
Gotcha, we should be all good now.
> @Chromosomologist would you please resolve conflicts on this pr? After some amount of struggle-- done ^^
Still need to get around to implementing a check to prevent messy inheritance, and I'm still undecided about what to do with duplicates. The other feedback should all be incorporated...