-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Refactor] Add new ExecutionContext class #109
Comments
Merged
ctsstc
changed the title
Add new ExecutionContext class
[Refactor] Add new ExecutionContext class
Apr 26, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I would like to have a defined structure to pass a context through to the execute commands. Not all commands necessarily use any or all of these so it should not be required on the
ICommand.execute()
interface to require these.It should have:
Forgot to submit this :0
Also had this garbage left 🤔 can't remember what it was :0
Maybe I was going to go off about a class or interface ¯_(ツ)_/¯
Blocks
The text was updated successfully, but these errors were encountered: