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

Remove spread in destructuring #9472

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

arturcarvalho
Copy link
Contributor

Hi there,

The goal of this PR is to remove "spread in destructuring" from the spread syntax compatibility table as mentioned in this issue.

Supporting information:
On the object literal proposal it's mentioned:

rest elements for array destructuring assignment
rest properties for object destructuring assignment

Destructuring is rest and not spread.

The error message when the rest element also hints at being rest and not spread destructuring:

image

Hope this helps.

@github-actions github-actions bot added the data:js Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript label Mar 15, 2021
Copy link
Collaborator

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

Perfect. Thank you, @arturcarvalho! 🎉

@ddbeck ddbeck merged commit 433db6e into mdn:master Mar 16, 2021
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Mar 16, 2021
@arturcarvalho arturcarvalho deleted the remove-spread-in-destructuring branch March 16, 2021 20:05
ddbeck added a commit that referenced this pull request Mar 18, 2021
* Start release notes for 2021-03-18 release

* Add release note for #9438

* Add release note for #9459

* Add release note for #9458

* Add release note for #9382

* Add release note for #9439

* Add release note for #9468

* Add release note for #9472

* Add release note for #9433

* Add release statistics
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:js Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants