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 from javascript/operators/spread.json #2309

Closed
jcsahnwaldt opened this issue Jun 14, 2018 · 1 comment
Closed
Labels
data:js Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript

Comments

@jcsahnwaldt
Copy link
Contributor

I think spread_in_destructuring should be removed from javascript/operators/spread.json. As far as I can tell, rest_in_objects in javascript/operators/destructuring.json describes the same feature and is included on the correct MDN page.

Details:

The compat table on MDN page "Spread syntax" has a line about "Spread in destructuring" (equivalent to section spread_in_destructuring in javascript/operators/spread.json), but destructuring is hardly mentioned in the text of the page.

I think the feature is described in section "Rest in Object Destructuring" on page "Destructuring assignment". The compat table on that page has a line about "Rest in objects", which is equivalent to rest_in_objects in javascript/operators/destructuring.json.

@Elchi3 Elchi3 added the data:js Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript label Jun 20, 2018
@ddbeck
Copy link
Collaborator

ddbeck commented Mar 16, 2021

Fixed by #9472.

@ddbeck ddbeck closed this as completed Mar 16, 2021
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

No branches or pull requests

3 participants