Skip to content

Create @babel/plugin-syntax-top-level-await#10573

Merged
nicolo-ribaudo merged 1 commit intobabel:masterfrom
nicolo-ribaudo:plugin-syntax-tla
Oct 29, 2019
Merged

Create @babel/plugin-syntax-top-level-await#10573
nicolo-ribaudo merged 1 commit intobabel:masterfrom
nicolo-ribaudo:plugin-syntax-tla

Conversation

@nicolo-ribaudo
Copy link
Copy Markdown
Member

Q                       A
Fixed Issues?
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature? Yes
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

As suggested by @hzoo in #10449 (comment), I added a caller flag for TLA used by preset-env, because both webpack and rollup already have experimental support for top level await.

The tests are currently failing because this PR depends on #10449

@nicolo-ribaudo nicolo-ribaudo added the PR: New Feature 🚀 A type of pull request used for our changelog categories label Oct 18, 2019
@nicolo-ribaudo nicolo-ribaudo added this to the v7.7.0 milestone Oct 18, 2019
@@ -0,0 +1,3 @@
node_modules
*.log
src
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung Oct 21, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: should also ignore test, to be future-proof.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just copy-pasted this file from somewhere else in the repo; we should fix many of them.

@nicolo-ribaudo nicolo-ribaudo added the PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release label Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: New Feature 🚀 A type of pull request used for our changelog categories PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release Spec: Top Level Await

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants