Skip to content

import(…, { with: { type: "json" } }) is standard track#25995

Merged
caugner merged 4 commits into
mdn:mainfrom
ddbeck:dynamic-import-assertion-standard
Feb 24, 2025
Merged

import(…, { with: { type: "json" } }) is standard track#25995
caugner merged 4 commits into
mdn:mainfrom
ddbeck:dynamic-import-assertion-standard

Conversation

@ddbeck

@ddbeck ddbeck commented Feb 21, 2025

Copy link
Copy Markdown
Contributor

Summary

The options argument for import() is just as standardized as the rest of import attributes.

Test results and supporting details

https://tc39.es/ecma262/multipage/ecmascript-language-expressions.html#sec-import-call-runtime-semantics-evaluation

Related issues

@github-actions github-actions Bot added data:js Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript size:xs [PR only] 0-6 LoC changed labels Feb 21, 2025
@ddbeck ddbeck changed the title import(…, { with: "json" }) is standard track import(…, { with: { type: "json" } }) is standard track Feb 21, 2025
Comment thread javascript/grammar.json
"status": {
"experimental": false,
"standard_track": false,
"standard_track": true,

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I set this as well, since it appears to be reflecting the same underlying fact.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done with 8ba99d2.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I guess there's no reason not to do it for both features. I've added it with 52ed946.

@github-actions github-actions Bot added size:s [PR only] 7-24 LoC changed and removed size:xs [PR only] 0-6 LoC changed labels Feb 22, 2025

@caugner caugner left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

image

@caugner caugner merged commit 1790762 into mdn:main Feb 24, 2025
@ddbeck ddbeck deleted the dynamic-import-assertion-standard branch February 24, 2025 18:56
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 size:s [PR only] 7-24 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants