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
Switch to yarn #84
Switch to yarn #84
Conversation
@@ -45,6 +45,7 @@ | |||
}, | |||
"dependencies": { | |||
"classnames": "^2.2.5", | |||
"eslint": "3.7.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's some comma-dangle
hick hack going on ... so we stick to that version for now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fspoettel don't you just thumbs up, give me a real review and approve!!! 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i thought i did, looks like i did not push the Approve button hard enough
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's ok, just make sure you are fully concentrated next time. It's important to stay focused when pressing buttons!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's currently one interesting discussion ongoing: yarnpkg/yarn#570
the behaviour at the moment is that when a normal
yarn install
is running, it automatically generates and updates theyarn.lock
file within the specified version range from thepackage.json
So in order to reproduce the correct dependencies always purely from the lock file it's important to add
--pure-lockfile
to theyarn install
command ... I hope they change that. But just in case people are wondering here why the hell the yarn.lock changes after an install.