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

make clearer where to add web-sys as dependency in Debugging chapter #206

Merged
merged 1 commit into from
Dec 9, 2020

Conversation

dlqs
Copy link
Contributor

@dlqs dlqs commented Jan 13, 2020

βœ‹ A similar PR may already be submitted!
Please search πŸ”Ž among the open pull requests before creating one.

Updating the Game of Life tutorial's code? Also send a PR to
rustwasm/wasm_game_of_life!

Now that you've checked, it's time to create your PR. πŸ“
Thanks for submitting! πŸ™

For more information, see the contributing guide. πŸ‘«

Summary

I made the mistake of adding web-sys twice, and also adding it right in the middle of the dependencies list, just like rustwasm/wasm-bindgen#1039.

I believe this small change makes it clearer where to add web-sys as a dependency with features.

@fitzgen fitzgen merged commit 05338e1 into rustwasm:master Dec 9, 2020
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.

2 participants