-
-
Notifications
You must be signed in to change notification settings - Fork 27k
Refactor Prototype pattern #1838
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
Conversation
SonarCloud Quality Gate failed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ThexXTURBOXx Thank you so much for your help and your interest to contribute to our repository.
However, we would want you to fix the Sonar issues that have been reported already before we take a look into your implementation.
Also make sure to add enough test cases to increase the code coverage to a minimum of 80%, though we always appreciate a full coverage of the same.
Looking forward to your contribution.
Cheers.
The pull request has remained inactive and is about to be closed. Please comment if you're still working on it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iluwatar I can pick this one up ☝️
Ok @yonatankarp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's open PR for this issue here: #1970 We can close this one...
Ok @yonatankarp, I'll close this one |
This PR refactors parts of the Prototype pattern by making it
Cloneable
and thus inheriting theclone()
method to its subclasses.