Skip to content

Commit

Permalink
chore: build.sh script unnecessarily requires lerna to be globally in…
Browse files Browse the repository at this point in the history
…stalled (aws#24217)

Either lerna needs to be installed as a dependency or run through npx. Otherwise build.sh fails in the middle of a build on a system without it installed.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
timmattison authored Apr 12, 2023
1 parent 923b9f1 commit 331622b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ concurrency=$(node -p 'Math.max(1, require("os").cpus().length - 1)')

echo "============================================================================================="
echo "building..."
time lerna run $bail --stream --concurrency=$concurrency $runtarget || fail
time npx lerna run $bail --stream --concurrency=$concurrency $runtarget || fail

if [ "$check_compat" == "true" ]; then
/bin/bash scripts/check-api-compatibility.sh
Expand Down

0 comments on commit 331622b

Please sign in to comment.