-
Notifications
You must be signed in to change notification settings - Fork 12.4k
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
added a new naming good practice #364
base: master
Are you sure you want to change the base?
added a new naming good practice #364
Conversation
kindly review this pr |
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.
Would be better to remove the good examples from the "bad" group to avoid confusion. For instance let anotherVariable = 0
is in both good and bad group. Same comment for AFunction
.
Thanks
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.
Good point 👍
Hey can you please merge this branch. After I make the required changes |
@thedivloop Kindly review the changes |
Use same type of casing style for same type of data in all the project