Skip to content

Commit d199363

Browse files
authored
Cleanup for actions and workflows (#570)
Light cleanup for actions and workflows. - Remove incorrect branch filters on `workflow_dispatch` triggers - Update `pr_workflow` filters - Add explicit read-level permissions where we checkout code Signed-off-by: Angela Burton <angelab@nvidia.com>
1 parent debbe7a commit d199363

5 files changed

Lines changed: 5 additions & 10 deletions

File tree

.github/workflows/all_libs.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ jobs:
1616
container: ghcr.io/nvidia/cuda-quantum-devcontainer:${{ matrix.platform }}-cu${{ matrix.cuda_version }}-gcc12-main
1717
permissions:
1818
actions: write
19+
contents: read
1920
pull-requests: read
2021
steps:
2122
- name: Checkout repository

.github/workflows/cudaq_cache.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,6 @@ name: CUDAQ cache
22

33
on:
44
workflow_dispatch:
5-
branches:
6-
- main
7-
85
push:
96
branches:
107
- main
@@ -56,4 +53,3 @@ jobs:
5653
save-build: true
5754
save-ccache: true
5855
platform: ${{ matrix.platform }}-cu${{ steps.config.outputs.cuda_major }}
59-

.github/workflows/docs.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,7 @@ name: Documentation
22

33
on:
44
workflow_call:
5-
65
workflow_dispatch:
7-
branches:
8-
- main
9-
106
push:
117
branches:
128
- main

.github/workflows/lib_qec.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ jobs:
1616
container: ghcr.io/nvidia/cuda-quantum-devcontainer:${{ matrix.platform }}-cu${{ matrix.cuda_version }}-gcc12-main
1717
permissions:
1818
actions: write
19+
contents: read
1920
pull-requests: read
2021
steps:
2122
- name: Checkout repository

.github/workflows/pr_workflow.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,8 @@ jobs:
4040
base: ${{ fromJSON(steps.get-pr-info.outputs.pr-info).base.sha }}
4141
filters: |
4242
build-cudaq:
43-
- '.github/workflows/cudaq_bump.yml'
4443
- '.github/actions/get-cudaq-build/**'
44+
- '.github/actions/get-cudaq-version/**'
4545
- '.github/workflows/scripts/build_cudaq.sh'
4646
- '.cudaq_version'
4747
build-docs:
@@ -53,13 +53,14 @@ jobs:
5353
build-all:
5454
- '.github/actions/build-lib/action.yaml'
5555
- '.github/actions/build-lib/build_all.sh'
56-
- '.github/actions/build-lib/build_all.yaml'
56+
- '.github/actions/build-lib/setup_custabilizer.sh'
5757
- '.github/workflows/all_libs.yaml'
5858
- 'cmake/Modules/**'
5959
- '**/CMakeLists.txt'
6060
build-qec:
6161
- '.github/actions/build-lib/action.yaml'
6262
- '.github/actions/build-lib/build_qec.sh'
63+
- '.github/actions/build-lib/setup_custabilizer.sh'
6364
- '.github/workflows/lib_qec.yaml'
6465
- 'cmake/Modules/**'
6566
- 'libs/core/**/*.cpp'

0 commit comments

Comments
 (0)