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

Add missing multiMemory WebAssembly feature feature #21620

Merged
merged 3 commits into from
Dec 21, 2023

Conversation

queengooborg
Copy link
Collaborator

This PR is a part of a project to add missing interfaces and interface features to BCD that are from an active spec (including WICG specs) and is supported in at least one browser. This particular PR adds the missing multiMemory WebAssembly feature. The data comes from the mdn-bcd-collector project (v10.6.3).

Check out the collector's guide on how to review this PR.

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/webassembly/multiMemory

This PR is a part of a project to add missing interfaces and interface features to BCD that are from an active spec (including WICG specs) and is supported in at least one browser. This particular PR adds the missing `multiMemory` WebAssembly feature. The data comes from the [mdn-bcd-collector](https://mdn-bcd-collector.gooborg.com) project (v10.6.3).

_Check out the [collector's guide on how to review this PR](https://github.com/openwebdocs/mdn-bcd-collector#reviewing-bcd-changes)._

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/webassembly/multiMemory
@queengooborg queengooborg added the data:wasm 🦾 Compat data for Web Assembly features. https://developer.mozilla.org/en-US/docs/WebAssembly label Dec 14, 2023
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

hm how do we deal with spec_urls for wasm again? They aren't in browser-specs, right?

I guess this is https://github.com/WebAssembly/multi-memory/blob/main/proposals/multi-memory/Overview.md?

@queengooborg
Copy link
Collaborator Author

Right now, we're just adding the links in as exceptions to the spec URL linter:

// Proposals for WebAssembly
'https://github.com/WebAssembly/spec/blob/main/proposals',
'https://github.com/WebAssembly/exception-handling/blob/main/proposals',
'https://github.com/WebAssembly/extended-const/blob/main/proposals',
'https://github.com/WebAssembly/tail-call/blob/main/proposals',
'https://github.com/WebAssembly/threads/blob/main/proposal',
'https://github.com/WebAssembly/relaxed-simd/blob/main/proposals',

@Elchi3
Copy link
Member

Elchi3 commented Dec 21, 2023

Ah okay, well I guess we should continue to add exception for new wasm features then.

@github-actions github-actions bot added the linter 🏡 Issues or pull requests regarding the tests / linter of the JSON files. label Dec 21, 2023
@Elchi3 Elchi3 merged commit 9e6d199 into mdn:main Dec 21, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:wasm 🦾 Compat data for Web Assembly features. https://developer.mozilla.org/en-US/docs/WebAssembly linter 🏡 Issues or pull requests regarding the tests / linter of the JSON files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants