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

import '...' with {...} #7179

Closed
arcanis opened this issue Mar 31, 2023 · 7 comments · Fixed by #7868
Closed

import '...' with {...} #7179

arcanis opened this issue Mar 31, 2023 · 7 comments · Fixed by #7868
Assignees
Milestone

Comments

@arcanis
Copy link

arcanis commented Mar 31, 2023

Describe the feature

The import '...' assert {...} syntax got changed into import '...' with {...} at the last tc39 meeting.

Babel plugin or link to the feature description

https://github.com/tc39/proposal-import-attributes

Additional context

The purpose of this change was only to explicitly allow runtimes to include the attributes in the module cache key (instead of just validating that the target module was matching the constraint), so it shouldn't need other changes in SWC other than adding support for the new syntax.

@nicolo-ribaudo
Copy link

Fyi, Babel is probably going to release support for this next week. I am also working on support for Rollup and Webpack (xtuc/acorn-import-attributes#21). It would be nice to have all the major tools supporting this in the next few months, so that the community can start migrating :)

@kdy1
Copy link
Member

kdy1 commented May 11, 2023

This is stage 2, so I'm not going to work on this.

@nicolo-ribaudo
Copy link

Its stage 3 pending reviews from other delegates to make sure that the spec text is correct. It already received two out of three reviews (tc39/proposal-import-attributes#137) — I'll ping again when the third one is done because then the proposal will automatically have real stage 3.

@lukastaegert
Copy link

@nicolo-ribaudo aren't the reviews done by now?

@kdy1 kdy1 self-assigned this Aug 25, 2023
@nicolo-ribaudo
Copy link

I'm sorry, I forgot to ping 😅

This is now been implemented in v8 (flagged) and JSC (unflagged), and is being implemented in TypeScript.

@kdy1
Copy link
Member

kdy1 commented Aug 25, 2023

I'll work on this today or tomorrow

kdy1 added a commit that referenced this issue Aug 28, 2023
@kdy1 kdy1 modified the milestones: Planned, v1.3.81 Aug 30, 2023
@swc-bot
Copy link
Collaborator

swc-bot commented Sep 29, 2023

This closed issue has been automatically locked because it had no new activity for a month. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@swc-project swc-project locked as resolved and limited conversation to collaborators Sep 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging a pull request may close this issue.

5 participants