-
-
Notifications
You must be signed in to change notification settings - Fork 26.6k
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
fix: Refactor Layered Architecture pattern #2936 #2938
Conversation
Quality Gate passedIssues Measures |
Let me know when it's ready for review |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Sorry, I have missed the notification that this PR is ready for review. If you could fix the conflicts then we are ready to merge @romannimets |
Quality Gate passedIssues Measures |
Sorry @iluwatar i may have made some mistakes during the conflict solving process 😅, i hope now everything is all right. Thaks for the patience |
Looks good! Thank you for the contribution 🎉 @all-contributors please add @romannimets for code |
I couldn't determine any contributions to add, did you specify any contributions? I've put up a pull request to add @romannimets! 🎉 |
What problem does this PR solve?
Ecco un possibile testo che potresti aggiungere nella tua pull request:
What problem does this PR solve?
This PR addresses a couple of specific issues in the Layered Architecture code example (#2936):
Main Entry Point for Runner.java: Runner.java lacked a main entry point, preventing it from being executed directly in an IDE. This PR adds a
main
method to Runner.java to resolve this issue.Warnings Regarding Optional Usage: There were warnings related to the usage of
Optional
as a type for theid
field in the classesCakeInfo
,CakeLayerInfo
, andCakeToppingInfo
. The use ofOptional
for fields is generally discouraged, asOptional
is primarily intended for method return types where there is a need to represent "no result" without using null.I'm relatively new to contributing and may have misunderstood or not fully addressed the issues. If there are any mistakes or if further changes are needed, please let me know. Apologies for any inconvenience caused.