Skip to content

Commit 741fe94

Browse files
yanxue06claude
andcommitted
fix(typescript-service-release): notify only on major (breaking) releases
The notify job fired on every real release (major | minor | patch). Per intent, release emails should only go out for breaking changes, so gate the job on release_type == 'major'. Minor/patch releases now skip the notify job cleanly. Updates the notify-on-release input description to match. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 719325e commit 741fe94

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/typescript-service-release.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ on:
7171
type: boolean
7272
required: false
7373
default: true
74-
description: "Fire the post-release notify job on real releases (major | minor | patch). Callers not on the in-job allow-list skip cleanly. Set to false to opt a specific release out."
74+
description: "Fire the post-release notify job on major (breaking) releases only. Minor/patch releases and callers not on the in-job allow-list skip cleanly. Set to false to opt a specific release out."
7575
secrets:
7676
OPENAI_API_KEY:
7777
required: true
@@ -233,7 +233,7 @@ jobs:
233233
(needs.npm-publish.result == 'success' || needs.npm-publish.result == 'skipped') &&
234234
(needs.npm-publish-oidc.result == 'success' || needs.npm-publish-oidc.result == 'skipped') &&
235235
inputs.notify-on-release &&
236-
contains(fromJSON('["major","minor","patch"]'), needs.release-info.outputs.release_type)
236+
needs.release-info.outputs.release_type == 'major'
237237
runs-on: ${{ inputs.use-blacksmith && 'blacksmith-4vcpu-ubuntu-2404' || 'ubuntu-latest' }}
238238
permissions:
239239
contents: read

0 commit comments

Comments
 (0)