From c858bb68ce4876b6709a0a75af39c924717d76ea Mon Sep 17 00:00:00 2001 From: Yifan Yang Date: Tue, 5 Dec 2023 12:17:39 -0800 Subject: [PATCH] Install missing dependencies during clean install by default Currently npx prompts for confirmation from user to install missing dependencies when running the script. However, there is now a use case (b/299490916) that requires the script to be run unattendedly. `--yes` suppresses the prompt (https://docs.npmjs.com/cli/v10/commands/npx). See b/299490916 and go/firebase-3p-frameworks-compatibility-testing for more details. --- scripts/clean-install.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/clean-install.sh b/scripts/clean-install.sh index c0cb19ef3ec..a33539ef86d 100755 --- a/scripts/clean-install.sh +++ b/scripts/clean-install.sh @@ -11,7 +11,7 @@ trap cleanup EXIT rm -rf ./clean || true echo "Running clean-publish --without-publish, as we would before publishing to npm..." -npx clean-publish --without-publish --before-script ./scripts/clean-shrinkwrap.sh --temp-dir clean +npx --yes clean-publish --without-publish --before-script ./scripts/clean-shrinkwrap.sh --temp-dir clean echo "Ran clean-publish --without-publish." echo "Packaging cleaned firebase-tools..." cd ./clean