Skip to content

Fix Travis CI npm install issues#1824

Merged
maurizi merged 1 commit intoOpenTreeMap:masterfrom
maurizi:fix-travis-npm
Dec 4, 2014
Merged

Fix Travis CI npm install issues#1824
maurizi merged 1 commit intoOpenTreeMap:masterfrom
maurizi:fix-travis-npm

Conversation

@maurizi
Copy link
Contributor

@maurizi maurizi commented Dec 3, 2014

travis-ci/travis-ci#2244 (comment)
mentions creating the npm directory manually as a temporary workaround

@maurizi maurizi changed the title Running npm install *occassionally* fails on Travis CI Running npm install occassionally fails on Travis CI Dec 3, 2014
@maurizi maurizi changed the title Running npm install occassionally fails on Travis CI Fix Travis CI npm install issues Dec 3, 2014
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) when pulling 873c546 on maurizi:fix-travis-npm into 11d1f83 on OpenTreeMap:master.

@hectcastro
Copy link
Contributor

Looking at some of the other fixes applied, this one (sublimator/rippled@bc4f368) seems like the most comprehensive (includes an additional chown step). That said, I still can't make sense of why either one fixes the issue, or why the issue only happens sometimes.

👍, because it appears that the issue went away. Leaving the comment above incase this issue resurfaces.

@maurizi
Copy link
Contributor Author

maurizi commented Dec 4, 2014

@hectcastro Thanks for the detective work. I'm going to go with the more comprehensive solution you linked to, because it seems like it can't hurt (and commenting weirdness is always a good plan)

travis-ci/travis-ci#2244 (comment)
mentions creating the npm directory manually as a temporary workaround
maurizi added a commit that referenced this pull request Dec 4, 2014
Fix Travis CI npm install issues
@maurizi maurizi merged commit f57d121 into OpenTreeMap:master Dec 4, 2014
@maurizi maurizi deleted the fix-travis-npm branch December 4, 2014 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants