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

Missing specification data spread syntax #12701

Closed
harshvchawla opened this issue Oct 4, 2021 · 0 comments · Fixed by #17328
Closed

Missing specification data spread syntax #12701

harshvchawla opened this issue Oct 4, 2021 · 0 comments · Fixed by #17328
Labels
data:js Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript

Comments

@harshvchawla
Copy link

image

What information was incorrect, unhelpful, or incomplete?

  1. spread.json contains json in the format of {javascript.operators.spread...} whereas spread is a syntax and not an operator
{
  "javascript": {
    "operators": {
      "spread": {
  1. All the spec_url values in the spread.json point to https://tc39.es/ecma262/multipage/ecmascript-language-expressions.html#sec-array-initializer

What did you expect to see

  1. Recommendation: Perhaps move spread.json out of javascript/operators folder (and into https://github.com/mdn/browser-compat-data/tree/main/javascript/builtins folder?)

  2. Recommendation: spec_url can instead point to https://tc39.es/ecma262/multipage/ecmascript-language-expressions.html#prod-SpreadElement

Did you test this? If so, how?

Searching spread in ecma specification yields the recommended spec_url

Can you link to any release notes, bugs, pull requests, or MDN pages related to this?

https://stackoverflow.com/a/44934830/2361131

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 a pull request may close this issue.

2 participants