Skip to content

Remove component names subsumed by external-id - #2735

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:remove-import-name-forms
Oct 5, 2026
Merged

alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:remove-import-name-forms

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

This commit removes support for component names of the form unlocked-dep=<...>, locked-dep=<...>, url=<...> and integrity=<...>. These were removed upstream in
WebAssembly/component-model#672 in favor of the external-id attribute/annotation. I'm not aware of anyone actually using these forms of imports anywhere in the ecosystem so the expected impact of this change in theory is negligible. If there are users, however, these can always be re-added with a more phased deletion plan.

This commit removes support for component names of the form
`unlocked-dep=<...>`, `locked-dep=<...>`, `url=<...>` and
`integrity=<...>`. These were removed upstream in
WebAssembly/component-model#672 in favor of the `external-id`
attribute/annotation. I'm not aware of anyone actually using these forms
of imports anywhere in the ecosystem so the expected impact of this
change in theory is negligible. If there are users, however, these can
always be re-added with a more phased deletion plan.
@alexcrichton
alexcrichton requested a review from a team as a code owner October 2, 2026 00:59
@alexcrichton
alexcrichton requested review from pchickey and removed request for a team October 2, 2026 00:59
@alexcrichton
alexcrichton added this pull request to the merge queue Oct 5, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 5, 2026
@alexcrichton
alexcrichton added this pull request to the merge queue Oct 5, 2026
Merged via the queue into bytecodealliance:main with commit 86f74cc Oct 5, 2026
37 checks passed
@alexcrichton
alexcrichton deleted the remove-import-name-forms branch October 5, 2026 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants