fix(ng-dev): prevent fallthrough for switch cases in target label checks

josephperrott · josephperrott · commit 270abd3a1abd · 2025-09-12T22:16:47.000Z
diff --git a/ng-dev/pr/common/validation/assert-allowed-target-label.ts b/ng-dev/pr/common/validation/assert-allowed-target-label.ts
@@ -73,6 +73,7 @@ class Validation extends PullRequestValidation {
         if (hasDeprecations && !releaseTrains.isFeatureFreeze()) {
           throw this._createHasDeprecationsError(targetLabel);
         }
+        break;
       case targetLabels['TARGET_AUTOMATION']:
         if (!automationBots.includes(pullRequest.author.login)) {
           throw this._createUserUsingAutomationLabelError(targetLabel, pullRequest.author.login);

Original file line number	Diff line number	Diff line change
`@@ -73,6 +73,7 @@ class Validation extends PullRequestValidation {`
`73`	`73`	`if (hasDeprecations && !releaseTrains.isFeatureFreeze()) {`
`74`	`74`	`throw this._createHasDeprecationsError(targetLabel);`
`75`	`75`	`}`
	`76`	`+ break;`
`76`	`77`	`case targetLabels['TARGET_AUTOMATION']:`
`77`	`78`	`if (!automationBots.includes(pullRequest.author.login)) {`
`78`	`79`	`throw this._createUserUsingAutomationLabelError(targetLabel, pullRequest.author.login);`