Skip to content
This repository was archived by the owner on Aug 25, 2025. It is now read-only.

Conversation

yokra9
Copy link

@yokra9 yokra9 commented Jun 21, 2020

console_error_panic_hook wasn't working properly, so I fixed Cargo.toml and src/lib.rs. ( #163 )

@kmaork
Copy link

kmaork commented Jul 25, 2020

Is there a reason this is not merged?

@jpryne
Copy link

jpryne commented Aug 1, 2020

I'd like to chime in that this issue arises in following the beginner tutorials for 'Getting Started with WASM-Pack':
https://rustwasm.github.io/wasm-pack/book/tutorials/hybrid-applications-with-webpack/getting-started.html

A confusing issue for beginners which will no doubt discourage some new users from continuing on their quest to learn Rust. Please merge! 👍

Copy link

@Herohtar Herohtar left a comment

Choose a reason for hiding this comment

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

This needs to be merged. The use of non-existent conditional flags doesn't work and creates confusion for beginners who are following the tutorials that use this.

Edit: Technically, the comments are incorrect, as they imply that compiling in release vs debug will result in different behavior.

yokra9 and others added 3 commits September 1, 2021 22:28
Co-authored-by: Herohtar <herohtar@tuta.io>
Co-authored-by: Herohtar <herohtar@tuta.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants