You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I randomly stumbled upon this repository and found an issue.
The yarn lockfile must be included in the repository, as it ensures that all dependencies and subdependencies are the same accross everyone's computers. That is why yarn was created in the first place to replace npm.
If the README instructs people to run npm run reinstall, is it possible this project relies on NPM instead of Yarn? Maybe the authors just wanted to make things as easy as possible, or perhaps there's a specific reason.
haslo
added a commit
to haslo/diskwarsbuilder
that referenced
this issue
May 7, 2018
Either way, there should be some sort of lock file. Maybe for acceptance we can have a package-lock.json as well. It's best practice to check the lock file in so that those who use the boilerplate have the exact versions of node_modules that a developer has while creating this and finding that things are working. As I worked on #68 and created a PR where I had to change package.json dependencies, I can fix this one too.
I randomly stumbled upon this repository and found an issue.
The yarn lockfile must be included in the repository, as it ensures that all dependencies and subdependencies are the same accross everyone's computers. That is why yarn was created in the first place to replace npm.
elm-webpack-starter/.gitignore
Line 5 in 039c6da
The flow should be:
yarn install
.yarn.lockfile
to the repository.This will ensure that everyone else have the exact same setup, allowing for easier tracking of bugs when they occur.
The text was updated successfully, but these errors were encountered: