This repository was archived by the owner on May 21, 2024. It is now read-only.
This repository was archived by the owner on May 21, 2024. It is now read-only.
Open
Description
The FrameworkContext docs state: this context type contains every data that's relevant to the command. Would it be possible to move the &Message parameter, that every command function currently had, into FrameworkContext as well?
- it would make command functions shorter and cleaner
- it would be more consistent (since other data relevant to the command already resides in FrameworkContext)
- it encapsulates all useful command context into a single struct that can be passed around easily (for example in my bot using this framework I will have to pass around both
FrameworkContextand&Messageto a certain utility function, which seems redundant)
Is there an issue with the proposal that I hadn't considered? Otherwise I'd gladly create a PR for this