zuul/zuul-jobs
Code Issues Proposed changes

Pin to npm4 until npm 5.6.0 comes out

Browse Source 
npm5 has an issue uploading tarballs it didn't build. For now, install
npm4 and use that to upload our tarballs.

The fix for https://github.com/npm/npm/issues/16723 should be in 5.6.0
which should allow us to revert this.

Change-Id: I8f633ee2119218799706b47ba3b471dcb04c1253
This commit is contained in:
Monty Taylor 2017-12-08 17:20:51 -06:00
parent 2862cddb6a
commit 0cd5075466
No known key found for this signature in database
GPG Key ID: 7BAE94BC7141A594
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
roles/upload-npm/tasks/main.yaml
View File

@ -20,4 +20,6 @@
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh" # This loads nvm
nvm install node
nvm use node
npm publish --ignore-scripts {{ zuul.executor.work_root }}/artifacts/{{ zuul.project.short_name }}-{{ project_ver }}.tar.gz
# Workaround for https://github.com/npm/npm/issues/16723
npm install npm@4
node_modules/.bin/npm publish --ignore-scripts {{ zuul.executor.work_root }}/artifacts/{{ zuul.project.short_name }}-{{ project_ver }}.tar.gz