Skip to content

Commit 62e67a8

Browse files
committed
Add more comments
1 parent 2c2f955 commit 62e67a8

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

.github/workflows/ok-to-test.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ on:
88
jobs:
99
ok-to-test:
1010
runs-on: ubuntu-latest
11+
# required permissions for adding reactions to the pull request comments
1112
permissions:
1213
pull-requests: write
1314
# Only run for PRs, not issue comments
@@ -20,4 +21,5 @@ jobs:
2021
reaction-token: ${{ secrets.GITHUB_TOKEN }}
2122
issue-type: pull-request
2223
commands: ok-to-test
24+
# The repository permission level required by the user to dispatch commands. Only allows 1Password collaborators to run this.
2325
permission: write

.github/workflows/validate.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,9 @@ jobs:
5353
# It will checkout the forked (and now trusted) code and it will run the integration tests on it.
5454
# If the tests are successful this action will proceed to update the status of the forked PR integration check.
5555
integration-test-fork:
56-
# must have these permissions to
56+
# required permissions for updating the status of the pull request checks
5757
permissions:
58-
pull-requests: write
58+
pull-request: write
5959
checks: write
6060
strategy:
6161
matrix:

0 commit comments

Comments
 (0)