search

scottwrobinson / camo

A class-based ES6 ODM for Mongo-like databases.
556 stars 80 forks source link

Use 'defaultValue' instead of 'default'. #45

Closed michaeljota closed 8 years ago

michaeljota commented 8 years ago

Because 'default' it's a reserved word.

scottwrobinson commented 8 years ago

I don't really see a problem with this since the schema default won't ever interfere with JavaScript's default keyword. It would only be a problem if you used it as a variable name.

Some linters, like jshint, do highlight it as a keyword, but not because it shouldn't be used, but because their highlighting it fairly naive.

I'm going to leave this as is.

Thanks!