Skip to content

Commit e088c42

Browse files
committed
Fix perms to repo secrets for integ tests
1 parent 34050f9 commit e088c42

File tree

4 files changed

+18
-8
lines changed

4 files changed

+18
-8
lines changed

.github/workflows/checks.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,3 @@ jobs:
1212
uses: ./.github/workflows/typescript-checks.yml
1313
cdk:
1414
uses: ./.github/workflows/cdk-checks.yml
15-
integ-test:
16-
uses: ./.github/workflows/integ-tests.yml

.github/workflows/integ-tests.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@ name: Integ Tests
33
on:
44
workflow_dispatch: {}
55
workflow_call:
6+
push:
7+
branches:
8+
- main
69

710
jobs:
811
integ_tests:
Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
11
name: Pull request checks
22

33
on:
4-
pull_request_target:
5-
types: [opened, synchronize, reopened, labeled]
6-
branches: [main]
4+
pull_request:
75

86
jobs:
97
python:
@@ -12,6 +10,3 @@ jobs:
1210
uses: ./.github/workflows/typescript-checks.yml
1311
cdk:
1412
uses: ./.github/workflows/cdk-checks.yml
15-
integ-test:
16-
if: contains(github.event.pull_request.labels.*.name, 'safe-to-test')
17-
uses: ./.github/workflows/integ-tests.yml
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
name: Pull request integration tests
2+
3+
on:
4+
pull_request_target:
5+
types: [opened, synchronize, reopened, labeled]
6+
branches: [main]
7+
8+
jobs:
9+
integ-test:
10+
if: contains(github.event.pull_request.labels.*.name, 'safe-to-test')
11+
permissions:
12+
id-token: write
13+
contents: read
14+
uses: ./.github/workflows/integ-tests.yml

0 commit comments

Comments
 (0)