-
Notifications
You must be signed in to change notification settings - Fork 129
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
Object Calisthenics - best practise #236
Conversation
manual/basic-guidelines.md
Outdated
8. No Classes With More Than Two Instance Variables | ||
9. No Getters/Setters/Properties | ||
|
||
for more info: |
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.
For more info:
This comment was marked as abuse.
This comment was marked as abuse.
always lookingg to find faults :( |
This comment was marked as abuse.
This comment was marked as abuse.
thank you for your words, these words really encourage |
This comment was marked as abuse.
This comment was marked as abuse.
nemesis |
If people like you who can develop code actually did instead of constantly
blaming those that try then joomla would be in a much better place.
…On Sat, 19 May 2018, 13:03 Nicola Galgano, ***@***.***> wrote:
Closed #236 <#236>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#236 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABPH8eDEITi5yt9e2_p6H2hIm3FNrx5jks5tz_vkgaJpZM4UFqHf>
.
|
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
reopened, just the time to dress an anti-gas mask |
Object Calisthenics is heavily opinionated and unenforceable in the Joomla code without extremely disruptive B/C breaks. So no, this can't just be blindly copy/pasted in and enforced as part of the coding standard. |
as per discussion on joomla/joomla-cms#20417 (comment)