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.

Sign up for GitHub

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

Jump to bottom

Fix: State pattern #503

Open

mth-bou wants to merge 3 commits into DesignPatternsPHP:main from mth-bou:fix/state-pattern

mth-bou commented Apr 18, 2024 •

edited

This pull request makes the following changes:

Fix issue Class diagram doesn't match the code. Pattern "State" #494

Checklist:

I've renamed the classes to match the UML diagram
I've added the return types for each method
I've updated the relevant README file

mth-bou added 3 commits

April 18, 2024 11:23


          fix: classes renaming to correspond to UML diagram

3a8af1c


          fix: add return types for all methods

0c8f10f


          fix: update README.rst

699ff11

Author

mth-bou commented Apr 18, 2024

Hi @BorislavSabev , this PR is ready for review, thanks for your work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment