Skip to content

Commit

Permalink
Print message when npm/yarn install fails during auto-indexing (#165)
Browse files Browse the repository at this point in the history
Previously, auto-indexing jobs silently ignored npm/yarn install
failures making it difficult to understand why the failures were
ignored.
  • Loading branch information
olafurpg authored Jul 28, 2022
1 parent 4c8ca65 commit eaad661
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion dev/lenient-npm.sh
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
#!/usr/bin/env bash
set -eux


if [ $# -gt 0 ] && [ "$1" == "install" ]; then
/usr/local/bin/actual-npm "$@" || true
/usr/local/bin/actual-npm "$@" || echo "scip-typescript: ignoring npm install failure, will try to auto-index the project with partial dependency information"
else
/usr/local/bin/actual-npm "$@"
fi
2 changes: 1 addition & 1 deletion dev/lenient-yarn.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
set -eux

if [ $# -gt 1 ] && [ "$1" == "install" ]; then
/usr/local/bin/actual-yarn "$@" || true
/usr/local/bin/actual-yarn "$@" || echo "scip-typescript: ignoring yarn install failure, will try to auto-index the project with partial dependency information"
else
/usr/local/bin/actual-yarn "$@"
fi

0 comments on commit eaad661

Please sign in to comment.