Skip to content

Commit c2b346e

Browse files
committed
✏️ Fix the bug at the trigger condition of the document release workflow.
1 parent cd3db43 commit c2b346e

File tree

1 file changed

+13
-8
lines changed

1 file changed

+13
-8
lines changed

.github/workflows/documentation.yaml

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ on:
44
push:
55
branches:
66
# - "develop/**" # For development and testing
7-
- master
7+
- "master"
88
paths:
99
# Documentation
1010
# Doc - CI workflow
@@ -46,12 +46,18 @@ jobs:
4646
# Guard job: Check if documentation files changed and if release workflow succeeded
4747
check_docs_changes:
4848
runs-on: ubuntu-latest
49-
# Only run for workflow_run events (not direct push events)
50-
if: github.event_name == 'workflow_run'
5149
outputs:
52-
should_deploy: ${{ steps.check_changes.outputs.should_deploy }}
50+
should_deploy: ${{ steps.check_changes.outputs.should_deploy || steps.set_push_deploy.outputs.should_deploy }}
5351
steps:
52+
- name: Set deployment flag for push events
53+
id: set_push_deploy
54+
if: github.event_name == 'push'
55+
run: |
56+
echo "Direct push event detected, proceeding with deployment"
57+
echo "should_deploy=true" >> $GITHUB_OUTPUT
58+
5459
- name: Download docs update flag from release workflow
60+
if: github.event_name == 'workflow_run'
5561
uses: actions/download-artifact@v5
5662
with:
5763
name: release-docs-flag
@@ -60,6 +66,7 @@ jobs:
6066

6167
- name: Check release workflow success and docs update flag
6268
id: check_changes
69+
if: github.event_name == 'workflow_run'
6370
run: |
6471
set -euo pipefail
6572
@@ -85,10 +92,8 @@ jobs:
8592
8693
deploy_documentation:
8794
runs-on: ubuntu-latest
88-
# Run for direct push OR workflow_run with docs changes
89-
if: |
90-
github.event_name == 'push' ||
91-
(github.event_name == 'workflow_run' && needs.check_docs_changes.outputs.should_deploy == 'true')
95+
# Run based on the check_docs_changes job output (which handles both push and workflow_run events)
96+
if: needs.check_docs_changes.outputs.should_deploy == 'true'
9297
needs: [check_docs_changes]
9398
steps:
9499
- name: Checkout

0 commit comments

Comments
 (0)