From d33bee7c6510775a84fd268662362ae22a423e8c Mon Sep 17 00:00:00 2001 From: Tyler Butler Date: Tue, 21 Mar 2023 10:54:31 -0700 Subject: [PATCH] fix(release): Handle released dependency bumps (#14669) The build tools did not handle the case where a pre-release dependency needs to be bumped. I re-used an existing prompt handler as a quick fix. If needed we can follow up with a better change. --- .../src/handlers/fluidReleaseStateHandler.ts | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/build-tools/packages/build-cli/src/handlers/fluidReleaseStateHandler.ts b/build-tools/packages/build-cli/src/handlers/fluidReleaseStateHandler.ts index c20023880903..c4b2b97b89ae 100644 --- a/build-tools/packages/build-cli/src/handlers/fluidReleaseStateHandler.ts +++ b/build-tools/packages/build-cli/src/handlers/fluidReleaseStateHandler.ts @@ -320,7 +320,8 @@ export class FluidReleaseStateHandler extends InitFailedStateHandler { break; } - case "PromptToPRDeps": { + case "PromptToPRDeps": + case "PromptToPRReleasedDepsBump": { result = await promptToPRDeps(state, machine, testMode, log, data); break; } @@ -343,17 +344,6 @@ export class FluidReleaseStateHandler extends InitFailedStateHandler { break; } - case "PromptToPRReleasedDepsBump": { - if (testMode) return true; - - log.errorLog(`Not yet implemented`); - if (data.exitFunc !== undefined) { - data.exitFunc(101); - } - - break; - } - case "PromptToRunMinorReleaseCommand": { result = await promptToRunMinorReleaseCommand(state, machine, testMode, log, data); break;