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

Upgrade Dependencies to Resolve FCW Warnings #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DTM9025
Copy link
Contributor

@DTM9025 DTM9025 commented Mar 3, 2025

Building this project started to emit future compatibility warnings. It appears to be related to these issues (rust-lang/rust#79813, rust-lang/rust#71871). We resolve this by upgrading wasm-bindgen to at least 0.2.89.

Unfortunately, upstream seed 0.6.0 is incompatible with that and upgrading seed is way too messy, so replace that package with a fork that just has the necessary patch. If there are security concerns with using my fork, this can be changed to a fork of your own, which can be replicated by checking out the upstream seed's 0.6.0 tag and cherry-picking this commit.

Building this project started to emit future compatibility warnings.
We resolve this by upgrading wasm-bindgen to at least 0.2.89.

Unfortunately upstream seed 0.6.0 is incompatible with that and
upgrading that is way too messy, so replace that package with a fork
that just has the necessary patch.
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.

1 participant