Skip to content

lets use pascal case #3

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

Open
jbkoh wants to merge 1 commit into
base: main
Choose a base branch
from
Open

lets use pascal case #3

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions meta/class_naming_convention.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Brick has a historical reason to use underscores between pascal-case words to easily separate them with rules. Given that now we are explicitly defining facets of classes, we can remove the underscores and use pure pascal-case for class names.

This will give much cleaner coding conventions and better readability for end users.

We can keep backwards compatibility for the old class names by creating aliases for them, but we should encourage users to switch to the new naming convention.