-
-
Notifications
You must be signed in to change notification settings - Fork 681
Rule proposal: private-property-names
#246
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
Comments
I would hold off on this one for a little bit, as I'd like Evan to first approve this convention when he reviews the style guide. |
@chrisvfritz do we know more now? |
I don't think we've had a definitive ruling yet, but I'll ping @yyx990803. 🙂 |
Still no word from Evan, but let's move forward on this anyway. I think
What do you think @michalsnik? |
According to the Style Guide we should warn about following keys:
_foo
$foo
and encourage to prefix private properties with
$_
(e.g.$_foo
).This rule could live alongside
no-reserved-keys
, but we might as well keep only this rule, to not duplicate warnings.The text was updated successfully, but these errors were encountered: