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

deps: Unify wasmparser usage to a single version #4426

Merged
merged 2 commits into from
Jan 31, 2024

Conversation

theduke
Copy link
Contributor

@theduke theduke commented Jan 31, 2024

Upgrade the wasmparser version used by all crates to 0.121.0, and switch
to a shared workspace dependency.

Closes #3729

@theduke theduke requested a review from syrusakbary as a code owner January 31, 2024 19:46
Upgrade the wasmparser version used by all crates to 0.121.0, and switch
to a shared workspace dependency.
@theduke theduke force-pushed the issue-3729-duplicate-deps branch from 7f1bb64 to bb9f246 Compare January 31, 2024 20:08
@theduke theduke enabled auto-merge (rebase) January 31, 2024 20:12
The module was copy-pasted between js and jsc backends.

This commit lifts the module up to the root and shares it.
@theduke theduke force-pushed the issue-3729-duplicate-deps branch from bb9f246 to 5165868 Compare January 31, 2024 20:21
@theduke theduke merged commit 290ab9f into master Jan 31, 2024
51 of 52 checks passed
@theduke theduke deleted the issue-3729-duplicate-deps branch January 31, 2024 21:14
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.

Use a single wasmparser version
2 participants