Skip to content
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

Implement parser support for destructuring-private #13999

Merged
merged 3 commits into from Feb 1, 2022

Conversation

JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Nov 25, 2021

This PR tracks parser support progress for https://github.com/tc39/proposal-destructuring-private.

Related #13889

@JLHwung JLHwung added PR: New Feature 🚀 A type of pull request used for our changelog categories umbrella ☂️ labels Nov 25, 2021
@babel-bot
Copy link
Collaborator

babel-bot commented Nov 25, 2021

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/51000/

@JLHwung JLHwung added this to the v7.17.0 milestone Nov 25, 2021
@JLHwung JLHwung force-pushed the feat-7.17.0/destructuring-private branch from c87850c to 05317f7 Compare November 26, 2021 21:16
@nicolo-ribaudo
Copy link
Member

I chatted with @JLHwung on Slack; since the transform plugin is not ready yet we can merge the parser support in 7.17.0 and defer the transform to a future minor.

@JLHwung JLHwung force-pushed the feat-7.17.0/destructuring-private branch from 05317f7 to d8c7a8a Compare February 1, 2022 19:05
@JLHwung JLHwung changed the title Implement support for destructuring-private Implement parser support for destructuring-private Feb 1, 2022
* feat: parse destructuring private

* add test cases

* fix: register private name on destructuring

* add typings

* add syntax plugin

* add generator test case

* add syntax plugin to preset-stage-2

* fix flow errors

* address review comments

* fix: use private name in toAssignable

* test: add case with undefined refExpressionErrors
@JLHwung JLHwung force-pushed the feat-7.17.0/destructuring-private branch from d8c7a8a to 269f49c Compare February 1, 2022 19:21
@JLHwung JLHwung force-pushed the feat-7.17.0/destructuring-private branch from f3f658a to 38ef0ab Compare February 1, 2022 19:39
@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 Feb 1, 2022
@nicolo-ribaudo nicolo-ribaudo merged commit b092bd0 into main Feb 1, 2022
@nicolo-ribaudo nicolo-ribaudo deleted the feat-7.17.0/destructuring-private branch February 1, 2022 23:22
@nicolo-ribaudo nicolo-ribaudo removed the PR: New Feature 🚀 A type of pull request used for our changelog categories label Feb 1, 2022
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label May 4, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 4, 2022
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: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants