-
Notifications
You must be signed in to change notification settings - Fork 438
Load dependencies from wasm-bindgen
#116
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks pretty good to me! I'll need to dig into it a bit more rather than a cursory look at the code for sure though.
src/manifest.rs
Outdated
}, | ||
} | ||
|
||
println!("{:?}", wasm); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this for debugging purposes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, indeed!
lol haha i was gonna implement this, but i guess i don't have to anymore, will review later today. |
This commit updates `wasm-pack` to read the `__wasm_pack_unstable` section of wasm binaries that it exeutes over. This section is then parsed and NPM dependencies are extracted, if any, and inserted into the generated `package.json`. Closes rustwasm#101
blocked pending RFC on npm deps expression |
i'm going to close this because it is very stale and we've decided that this is not how we want to load npm deps. |
This commit updates
wasm-pack
to read the__wasm_pack_unstable
section ofwasm binaries that it exeutes over. This section is then parsed and NPM
dependencies are extracted, if any, and inserted into the generated
package.json
.Closes #101