10000 Fix PR test workflow for forks · deep-soft/github-script@58e7e1d · GitHub
[go: up one dir, main page]

Skip to content

Commit 58e7e1d

Browse files
committed
Fix PR test workflow for forks
When the workflow is triggered from a fork the permissions for the `GITHUB_TOKEN` are read-only. This chane skips the write operations if we're running from a fork. See: https://docs.github.com/en/actions/reference/events-that-trigger-workflows#pull-request-events-for-forked-repositories
1 parent b251c3e commit 58e7e1d

File tree

1 file changed

+17
-17
lines changed

1 file changed

+17
-17
lines changed

.github/workflows/pull-request-test.yml

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -20,22 +20,22 @@ jobs:
2020
issue_number: context.payload.number,
2121
})
2222
23-
// Find any comment already made by the bot.
24-
const botComment = comments.find(comment => comment.user.id === 41898282)
25-
const commentBody = "Hello from actions/github-script! (${{ github.sha }})"
26-
27-
if (botComment) {
28-
await github.issues.updateComment({
29-
owner: context.repo.owner,
30-
repo: context.repo.repo,
31-
comment_id: botComment.id,
32-
body: commentBody
33-
})
23+
if (context.payload.pull_request.head.repo.full_name !== 'actions/github-script') {
24+
console.log('Not attempting to write comment on PR from fork');
3425
} else {
35-
await github.issues.createComment({
36-
owner: context.repo.owner,
37-
repo: context.repo.repo,
38-
issue_number: context.payload.number,
39-
body: commentBody
40-
})
26+
if (botComment) {
27+
await github.issues.updateComment({
28+
owner: context.repo.owner,
29+
repo: context.repo.repo,
30+
comment_id: botComment.id,
31+
body: commentBody
32+
})
33+
} else {
34+
await github.issues.createComment({
35+
owner: context.repo.owner,
36+
repo: context.repo.repo,
37+
issue_number: context.payload.number,
38+
body: commentBody
39+
})
40+
}
4141
}

0 commit comments

Comments
 (0)
0