Skip to content

Commit f5a9f0c

Browse files
committed
Fix v6 release workflow branch targeting
1 parent af2cb8f commit f5a9f0c

File tree

3 files changed

+20
-15
lines changed

3 files changed

+20
-15
lines changed

.github/workflows/release-create-pr.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,12 @@ jobs:
3131

3232
steps:
3333
- uses: actions/checkout@v4
34+
- name: Validate release branch
35+
run: |
36+
if [ "${{ github.ref_name }}" != "v6.x" ]; then
37+
echo "This workflow must run from v6.x (current ref: ${{ github.ref_name }})."
38+
exit 1
39+
fi
3440
- uses: actions/setup-node@v4
3541
with:
3642
node-version: '20'
@@ -52,6 +58,6 @@ jobs:
5258
uses: actions/github-script@v7
5359
with:
5460
script: |
55-
const defaultBranch = "${{ github.event.repository.default_branch }}"
61+
const releaseBranch = "${{ github.ref_name }}"
5662
const versionTag = "v${{ steps.bump.outputs.version }}"
57-
await require('./scripts/release').generatePr({ github, context, defaultBranch, versionTag })
63+
await require('./scripts/release').generatePr({ github, context, releaseBranch, versionTag })

.github/workflows/release.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@ name: Create release
33
on:
44
push:
55
branches:
6-
- main
7-
- v6.x
6+
- v6.x
87
paths:
98
- package.json
109

@@ -41,7 +40,7 @@ jobs:
4140
release:
4241
runs-on: ubuntu-latest
4342
needs: check-release-version
44-
if: ${{ startsWith(needs.check-release-version.outputs.release-version, 'v') }}
43+
if: ${{ github.ref_name == 'v6.x' && startsWith(needs.check-release-version.outputs.release-version, 'v') }}
4544

4645
permissions:
4746
contents: write
@@ -66,6 +65,6 @@ jobs:
6665
uses: actions/github-script@v7
6766
with:
6867
script: |
69-
const defaultBranch = "${{ github.event.repository.default_branch }}"
68+
const releaseBranch = "${{ github.ref_name }}"
7069
const versionTag = "${{ needs.check-release-version.outputs.release-version }}"
71-
await require('./scripts/release').release({ github, context, defaultBranch, versionTag })
70+
await require('./scripts/release').release({ github, context, releaseBranch, versionTag })

scripts/release.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
// Called from .github/workflows
44

5-
const generateReleaseNotes = async ({ github, owner, repo, versionTag, defaultBranch }) => {
5+
const generateReleaseNotes = async ({ github, owner, repo, versionTag, releaseBranch }) => {
66
const { data: releases } = await github.rest.repos.listReleases({
77
owner,
88
repo
@@ -14,7 +14,7 @@ const generateReleaseNotes = async ({ github, owner, repo, versionTag, defaultBr
1414
owner,
1515
repo,
1616
tag_name: versionTag,
17-
target_commitish: defaultBranch,
17+
target_commitish: releaseBranch,
1818
previous_tag_name: previousRelease?.tag_name
1919
})
2020

@@ -25,29 +25,29 @@ const generateReleaseNotes = async ({ github, owner, repo, versionTag, defaultBr
2525
return bodyWithoutReleasePr
2626
}
2727

28-
const generatePr = async ({ github, context, defaultBranch, versionTag }) => {
28+
const generatePr = async ({ github, context, releaseBranch, versionTag }) => {
2929
const { owner, repo } = context.repo
30-
const releaseNotes = await generateReleaseNotes({ github, owner, repo, versionTag, defaultBranch })
30+
const releaseNotes = await generateReleaseNotes({ github, owner, repo, versionTag, releaseBranch })
3131

3232
await github.rest.pulls.create({
3333
owner,
3434
repo,
3535
head: `release/${versionTag}`,
36-
base: defaultBranch,
36+
base: releaseBranch,
3737
title: `[Release] ${versionTag}`,
3838
body: releaseNotes
3939
})
4040
}
4141

42-
const release = async ({ github, context, defaultBranch, versionTag }) => {
42+
const release = async ({ github, context, releaseBranch, versionTag }) => {
4343
const { owner, repo } = context.repo
44-
const releaseNotes = await generateReleaseNotes({ github, owner, repo, versionTag, defaultBranch })
44+
const releaseNotes = await generateReleaseNotes({ github, owner, repo, versionTag, releaseBranch })
4545

4646
await github.rest.repos.createRelease({
4747
owner,
4848
repo,
4949
tag_name: versionTag,
50-
target_commitish: defaultBranch,
50+
target_commitish: releaseBranch,
5151
name: versionTag,
5252
body: releaseNotes,
5353
draft: false,

0 commit comments

Comments
 (0)