-
-
Notifications
You must be signed in to change notification settings - Fork 158
Add stage-3 plugin option #384
Conversation
I think |
f44d5a6
to
fefd2b1
Compare
fefd2b1
to
3d54191
Compare
@@ -238,5 +237,14 @@ | |||
"ios": "10", | |||
"opera": "45", | |||
"electron": "1.7" | |||
} | |||
}, | |||
"transform-async-generator-functions": {}, |
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, optional catch binding, and Promise.finally will get filled out once we land #402
|
||
if (!plugin) { | ||
throw new Error( | ||
`Could not find plugin "${pluginName}". Ensure there is an entry in ./available-plugins.js for it.`, |
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 kept tripping me up 😜
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.
lol
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.
LGTM!
Also, good to update README.
And maybe we need to think a bit more about naming? Maybe includeStage3
will be more clear? I don't know :)
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 would be good, just means we can't have fixed versioning since stage 3 will change
I'll create a followup to update README, but we should def discuss if there's a better name for the option too. |
Adds support for enabling transformation of shipping stage 3 features.
EDIT Updating this to: