@samitouri / QOS-React / commits / ee6358bb7d

Tweak RRM instructions to mention Yarn instead

Dan Abramov committed Jan 23, 2017 at 18:01 UTC ee6358bb7dddab42256092545041ef621ad2e131
1 file changed +5 -8
scripts/release-manager/Readme.md
+5 -8
@@ -186,26 +186,23 @@ This will merge the commits you cherry-picked into the stable branch.
186
187 However, if you plan to cut an alpha or a beta, you should stay on the “stable development” branch.
188
189 -### Update the Shrinkwrap
189 +### Update the Lockfile
190
191 Run this so that the build is reproducible:
192
193 ```
194 -rm npm-shrinkwrap.json
194 +rm yarn.lock
195 rm -rf node_modules
196 -npm cache clear
197 -npm i
198 -npm shrinkwrap --dev
196 +yarn cache clean
197 +yarn
198 ```
199
200 Check `git diff`. Do changes look sensible?
201
203 -**Tip:** there are buggy npm versions that remove information from shrinkwrap. If you see a lot of `"dev": true` lines removed, you may be on a buggy version. Last time I tried, `npm@3.10.8` worked well.
204 -
202 Commit your changes:
203
204 ```
208 -git commit -am 'Update shrinkwrap'
205 +git commit -am 'Update Yarn lockfile'
206 ```
207
208 If you’re feeling extra careful, you can run `npm test` again.