-
-
Notifications
You must be signed in to change notification settings - Fork 234
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
feat: Enable classProperties and classPrivateProperties parsers. #379
Conversation
@bcoe I'd like to make sure this is in the |
/* Verify and add upon release of node.js 12: | ||
* classProperties | ||
* classPrivateProperties | ||
* classPrivateMethods |
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.
It seems classPrivateMethods are not supported in node.js v12.0.0. Will be keeping an eye on the v8 release blog and add plugins when appropriate.
} | ||
|
||
export function isClassPrivatePropAvailable() { | ||
return tryThis('class Foo { #a = 1; }', 'class private property'); |
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.
oh my goodness, I'm unreasonably excited about this feature.
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.
👍 this is awesome! private class properties are available now?
Yes in Chrome 74 and Node.js 12. Private getters/setters |
No description provided.