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

chore: dependency upgrade #32

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Suyashtnt
Copy link

This dependency upgrade is mainly required for cloudflare workers, where if you are using a version newer than 0.0.12, there is a linker error. This has been majorly untested though so I need to test it before marking it ready

@psarna
Copy link
Contributor

psarna commented Jul 24, 2023

@Suyashtnt thanks! I also experienced linking errors when updating dependencies in my demo (https://github.com/psarna/country-counter/), and traced it to rustwasm/wasm-bindgen#3276, which I worked around with freezing the dependency to =0.2.83: https://github.com/psarna/country-counter/blob/8817345361c40de26346bf6046c8c526be439108/Cargo.toml#L16C49-L16C101 . The wasm-pack issue is reportedly fixed though, so maybe bumping the dependencies already gets rid of the error

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.

None yet

2 participants