Skip to content

Commit

Permalink
Adding "|| exit 0" to prevent downstream failure in npm processes
Browse files Browse the repository at this point in the history
Various environmental or project scenarios may lead to a failure when calling lightercollective from postinstall. This is not known to be a lightercollective failure, but instead of a failure to properly call the lightercollective library. One such known scenario on Windows being multiple path casings being passed to child_process.spawn when npm-lifecycle processes postinstall.

Discussion on lifecycle/postinstall Win32 issue here: npm/npm-lifecycle#29

This PR is intended to address this discussion: WebReflection#165
  • Loading branch information
mattezell authored Apr 19, 2019
1 parent f50fb6d commit 1a0271b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
"scripts": {
"test": "phantomjs testrunner.js",
"web": "tiny-cdn run -p=1337",
"postinstall": "lightercollective"
"postinstall": "lightercollective || exit 0"
},
"devDependencies": {
"html-class": "^1.2.0",
Expand Down

0 comments on commit 1a0271b

Please sign in to comment.