Skip to content
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

Sugs* are commands? #4830

Open
Ducasse opened this issue Oct 4, 2019 · 3 comments

Comments

@Ducasse
Copy link
Member

commented Oct 4, 2019

I would like to know if I'm correct when I think that Sugs classes (besides the Sugs Context subclasses) are in fact commands.

I need a second pair of eyes to confirm my analysis.

@Ducasse

This comment has been minimized.

Copy link
Member Author

commented Oct 5, 2019

Julien if you have a moment. I'm interested in your point of view.

@juliendelplanque

This comment has been minimized.

Copy link
Member

commented Oct 5, 2019

I just had a look at it and it seems it is the implementation of a command design pattern. :-)

@Ducasse

This comment has been minimized.

Copy link
Member Author

commented Oct 5, 2019

:) Ok I thought the same. It looks like this is just the 26th one that we have :).
I will check to see if as a Kata I can replace them by CM2 commands.
:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.