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

Added example for weather report using rust and webassembly #2216

Merged
merged 1 commit into from Jun 25, 2020

Conversation

ayushmishra2005
Copy link
Contributor

Added new example using Rust and webassembly

Copy link
Contributor

@alexcrichton alexcrichton left a comment

Choose a reason for hiding this comment

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

Nice! Always happy to have more examples :)

examples/weather_report/Cargo.toml Outdated Show resolved Hide resolved
examples/weather_report/Cargo.toml Outdated Show resolved Hide resolved
examples/weather_report/Cargo.toml Outdated Show resolved Hide resolved
examples/weather_report/README.md Outdated Show resolved Hide resolved
examples/weather_report/src/lib.rs Show resolved Hide resolved
@ayushmishra2005
Copy link
Contributor Author

@alexcrichton I don't know why build is failing. Its running fine on local. copy-webpack-plugin is already there in package.json.

Please let me know if I am missing anything

@alexcrichton
Copy link
Contributor

Sorry I don't know much about npm, if it's failing and everything else is passing then I'm not entirely sure what's going on.

@alexcrichton
Copy link
Contributor

Apart from the build error though this looks good to go!

Minor refactoring

Updated readme file

Updated readme file

fixed build error

Fixed build error
@ayushmishra2005
Copy link
Contributor Author

ayushmishra2005 commented Jun 25, 2020

@alexcrichton I have fixed build issue. Please review and merge. Build is green now

@alexcrichton
Copy link
Contributor

Thanks!

@alexcrichton alexcrichton merged commit 979f0d2 into rustwasm:master Jun 25, 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.

None yet

2 participants