Skip to content

Commit 517d2a4

Browse files
committed
fix needs: groups, add changed-files guard for docs build
1 parent e4f4d19 commit 517d2a4

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

.github/workflows/build.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -267,6 +267,7 @@ jobs:
267267
build_type: ${{ inputs.build_type || 'branch' }}
268268
matrix_name: docs-build
269269
docs-build:
270+
needs: [docs-build-matrix]
270271
if: github.ref_type == 'branch'
271272
permissions:
272273
actions: read

.github/workflows/pr.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -743,9 +743,10 @@ jobs:
743743
contents: read
744744
uses: rapidsai/shared-workflows/.github/workflows/compute-matrix.yaml@release/26.10
745745
with:
746-
build_type: ${{ inputs.build_type || 'branch' }}
746+
build_type: pull-request
747747
matrix_name: docs-build
748748
docs-build:
749+
needs: [docs-build-matrix]
749750
permissions:
750751
actions: read
751752
contents: read
@@ -754,6 +755,7 @@ jobs:
754755
pull-requests: read
755756
secrets: inherit # zizmor: ignore[secrets-inherit]
756757
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@release/26.10
758+
if: fromJSON(needs.changed-files.outputs.changed_file_groups).build_docs
757759
strategy:
758760
fail-fast: false
759761
matrix: ${{ fromJSON(needs.docs-build-matrix.outputs.matrix) }}

0 commit comments

Comments
 (0)