Skip to content

Commit 45f3ae5

Browse files
authored
fix(repo): Fix Major Version Check action (#4355)
1 parent bf83dd5 commit 45f3ae5

File tree

2 files changed

+17
-3
lines changed

2 files changed

+17
-3
lines changed

.changeset/beige-spoons-type.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
---
2+
---

.github/workflows/major-version-check.yml

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ jobs:
2020
uses: actions/github-script@v7
2121
with:
2222
script: |
23-
const prNumber = context.payload.pull_request.number;
23+
const prNumber = context.payload?.pull_request?.number || context.payload?.issue?.number;
2424
2525
// Get list of files changed in the PR
2626
const { data: files } = await github.rest.pulls.listFiles({
@@ -29,6 +29,18 @@ jobs:
2929
pull_number: prNumber,
3030
});
3131
32+
let pullRequest = context.payload.pull_request;
33+
34+
if (!pullRequest) {
35+
// Fetch the pull request data
36+
const pullRequestData = await github.rest.pulls.get({
37+
owner: context.repo.owner,
38+
repo: context.repo.repo,
39+
pull_number: prNumber,
40+
});
41+
pullRequest = pullRequestData.data;
42+
}
43+
3244
// Check if any changeset files indicate a major bump
3345
let hasMajorChangeset = false;
3446
for (const file of files) {
@@ -38,7 +50,7 @@ jobs:
3850
owner: context.repo.owner,
3951
repo: context.repo.repo,
4052
path: file.filename,
41-
ref: context.payload.pull_request.head.sha,
53+
ref: pullRequest.head.sha,
4254
});
4355
4456
const content = Buffer.from(changesetContent.content, changesetContent.encoding).toString();
@@ -58,7 +70,7 @@ jobs:
5870
uses: actions/github-script@v7
5971
with:
6072
script: |
61-
const prNumber = context.payload.pull_request.number;
73+
const prNumber = context.payload?.pull_request?.number || context.payload?.issue?.number;
6274
const org = context.repo.owner;
6375
6476
// Get all comments on the PR

0 commit comments

Comments
 (0)