Skip to content
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

Naming conventions: camelCase vs separate-words #13

Open
carlesjove opened this issue Mar 5, 2015 · 5 comments
Open

Naming conventions: camelCase vs separate-words #13

carlesjove opened this issue Mar 5, 2015 · 5 comments
Labels
Milestone

Comments

@carlesjove
Copy link
Contributor

In issue #12 , a subtle decision is introduced: the use of camelCase for multiple word properties. Maybe it'd be a good idea to have some naming conventions at the spec, so authors of extensions follow it and clients know exactly what to expect.

@carlesjove
Copy link
Contributor Author

I vote for separate-words

@mamund
Copy link
Member

mamund commented Mar 6, 2015

i'd like to keep as much fidelity between internal code modeling and external message modeling. this is not a requirement, but something i'd like to keep an eye upon.

i prefer not using dashed names since it adds some "cruft" in coding for the properties. while JS allows this using brackets, i know of no other language that supports "dashes" in property/variable names.

  1. read-only
  2. readOnly
  3. readonly

are the basic options. i'd be ok w/ 2 or 3.

and i understand we're dealing with style here and setting a precedent. it's worth discussing.

comments?

@mamund
Copy link
Member

mamund commented Mar 11, 2015

it's been close to a week since i responded to this item. i'd like to close this up in order to move on teh actual implementation of the "read only" property marking for items.

does anyone want to actually discuss this?

@mamund mamund modified the milestone: cleanup Mar 11, 2015
@carlesjove
Copy link
Contributor Author

I'd just add that camelCase seems the best option to me, after what you
said.

On Wednesday, March 11, 2015, Mike Amundsen [email protected]
wrote:

it's been close to a week since i responded to this item. i'd like to
close this up in order to move on teh actual implementation of the "read
only" property marking for items.

does anyone want to actually discuss this?


Reply to this email directly or view it on GitHub
#13 (comment).

@mamund
Copy link
Member

mamund commented Mar 11, 2015

@carlesjove

ok, thanks.

unless i hear more on this, i'll write up some notes in the extensions section advising the preference for camelCase in all future extensions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants