This repository has been archived by the owner on May 14, 2021. It is now read-only.

Don't fail on npm outdated returning 1 #8

Closed

donaldguy wants to merge 1 commit into poise:master from tulip:master

donaldguy commented May 15, 2017

Simpler but bigger hammer version of #4

Manually tested on (an old dusty corner of) my prod:

can downgrade npm 4.5.0 -> 4.4.0
can reupgrade
can run multiple chef runs between without panics

if @coderanger doesn't want to merge (which is legit), there is a halite built versions you can get to by sticking in your Policyfile.rb or Berksfile

cookbook 'poise-javascript', github: 'tulip/poise-javascript', branch: 'built'

I'm not particularly planning to try to keep up to date, as our chef use at tulip is (sadly) diminishing (thanks k8s)


          Don't fail on npm outdated returning 1

9e8f9f0

donaldguy closed this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet