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

Fix runtime errors with upgrade to [email protected] #1

Merged
merged 1 commit into from
Jun 16, 2017

Conversation

jaxrtech
Copy link
Contributor

First off, hello! I had just ran across Whitestorm.js the other day and looks really promising.

I was trying to get started with this boilerplate/starter-kit project, but unfortunately, it wasn't working right off of the initial clone. It looks like the latest release of [email protected] on npm made breaking changes and broke the build. (Keeping in mind the existing package.json had ^2.0.0-beta.8 which will automatically match the latest ^2.0.0-beta-9.1 in terms of how semver matches versions using the caret).

Those issues have since been resolved and whs has been upgraded to beta-9.1 after cross-referencing examples from the framework's project.

I've also added a yarn.lock to hopefully prevent future dependency issues (given that the framework's project also includes one).

Thanks!

The `package.json` was fetching `2.0.0-beta.9.1` since it was the
latest in terms of a semver match.

Fixes breaking changes from `2.0.0-beta.8` that were keeping the
starter pack from running "out of the box".

Adds `yarn.lock` to prevent future dependency issues.
@sasha240100
Copy link
Member

@jaxrtech Thanks for the contribution! Let's wait for @hirako2000 to merge this PR.

@hirako2000 hirako2000 merged commit f76c3e7 into WhitestormJS:master Jun 16, 2017
@hirako2000
Copy link
Contributor

thanks @jaxrtech for helping out. Let's add the yarn file for now. Eventually we will move to npm5 and use its (new) lock system.

@jaxrtech
Copy link
Contributor Author

Thanks, sounds good! Glad to help! Yeah, it would make the sense to eventually move to the new npm lock system since the majority of devs use npm.

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.

3 participants