Skip to content

[parser] Enable "exportNamespaceFrom" by default#10521

Merged
nicolo-ribaudo merged 1 commit intobabel:masterfrom
nicolo-ribaudo:export-ns-from
Oct 29, 2019
Merged

[parser] Enable "exportNamespaceFrom" by default#10521
nicolo-ribaudo merged 1 commit intobabel:masterfrom
nicolo-ribaudo:export-ns-from

Conversation

@nicolo-ribaudo
Copy link
Copy Markdown
Member

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

It has been merged to the main spec: tc39/ecma262#1174

@nicolo-ribaudo nicolo-ribaudo added PR: New Feature 🚀 A type of pull request used for our changelog categories pkg: parser Spec: export-ns-from spec-update labels Oct 3, 2019
'estree' |
'exportDefaultFrom' |
'exportNamespaceFrom' |
'exportNamespaceFrom' | // deprecated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I wonder if a /** @deprecated */ annotation preceding the literal would work, or if it only works with enums

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'll check 👍

Copy link
Copy Markdown
Contributor

@JLHwung JLHwung left a comment

Choose a reason for hiding this comment

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

Could you also remove exportNamespaceFrom.syntax on packages/babel-core/src/transformation/util/missing-plugin-helper.js?

@nicolo-ribaudo
Copy link
Copy Markdown
Member Author

No, it's otherwise people using new @babel/core versions with an older parser won't get the nice error.

@nicolo-ribaudo nicolo-ribaudo added this to the v7.7.0 milestone Oct 12, 2019
@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 12, 2019
@nicolo-ribaudo nicolo-ribaudo merged commit 4cd93ab into babel:master Oct 29, 2019
@nicolo-ribaudo nicolo-ribaudo deleted the export-ns-from branch October 29, 2019 17:44
@lock lock Bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Jan 28, 2020
@lock lock Bot locked as resolved and limited conversation to collaborators Jan 28, 2020
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 pkg: parser 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: export-ns-from spec-update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants