Skip to content

Commit 3ae6074

Browse files
committed
change step order
1 parent 4e98be7 commit 3ae6074

File tree

1 file changed

+30
-29
lines changed

1 file changed

+30
-29
lines changed

.github/workflows/build-doc.yml

Lines changed: 30 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ jobs:
3434
run: |
3535
cd doc
3636
source .venv/bin/activate
37+
uv pip uninstall plotly
3738
uv pip install -e ..
3839
3940
- name: List installed packages
@@ -61,19 +62,6 @@ jobs:
6162
name: doc-html
6263
path: doc/build/html/
6364

64-
- name: Build API docs
65-
if: github.ref_name == 'migrate-to-gh-actions' && github.event_name == 'push'
66-
run: |
67-
cd doc
68-
source .venv/bin/activate
69-
# For the API doc, we need to use the local version of plotly
70-
# since we are tweaking the source because of
71-
# graph_objs/graph_objects
72-
uv pip uninstall plotly
73-
uv pip install -e ..
74-
cd apidoc
75-
make html
76-
7765
# Deploy steps — currently using test branches for validation
7866
# TODO: Change branch trigger to 'doc-prod' and deploy branches to 'built', 'built_ipynb', 'gh-pages'
7967
- name: Create GitHub App token
@@ -126,6 +114,35 @@ jobs:
126114
git commit -m "build of https://github.com/plotly/plotly.py/commit/${{ github.sha }}" || echo "No changes to commit"
127115
git push --force
128116
117+
- name: Checkout graphing-library-docs
118+
if: github.ref_name == 'migrate-to-gh-actions' && github.event_name == 'push'
119+
uses: actions/checkout@v6
120+
with:
121+
repository: plotly/graphing-library-docs
122+
ref: test-trigger
123+
token: ${{ steps.app-token.outputs.token }}
124+
path: graphing-library-docs
125+
126+
- name: Trigger downstream doc build
127+
if: github.ref_name == 'migrate-to-gh-actions' && github.event_name == 'push'
128+
run: |
129+
cd graphing-library-docs
130+
git commit --allow-empty -m "deploying https://github.com/plotly/plotly.py/commit/${{ github.sha }}"
131+
git push
132+
133+
- name: Build API docs
134+
if: github.ref_name == 'migrate-to-gh-actions' && github.event_name == 'push'
135+
run: |
136+
cd doc
137+
source .venv/bin/activate
138+
# For the API doc, we need to use the local version of plotly
139+
# since we are tweaking the source because of
140+
# graph_objs/graph_objects
141+
uv pip uninstall plotly
142+
uv pip install -e ..
143+
cd apidoc
144+
make html
145+
129146
- name: Checkout plotly.py-docs (gh-pages-test)
130147
if: github.ref_name == 'migrate-to-gh-actions' && github.event_name == 'push'
131148
uses: actions/checkout@v6
@@ -145,19 +162,3 @@ jobs:
145162
git add .
146163
git commit -m "build of https://github.com/plotly/plotly.py/commit/${{ github.sha }}" || echo "No changes to commit"
147164
git push --force
148-
149-
- name: Checkout graphing-library-docs
150-
if: github.ref_name == 'migrate-to-gh-actions' && github.event_name == 'push'
151-
uses: actions/checkout@v6
152-
with:
153-
repository: plotly/graphing-library-docs
154-
ref: test-trigger
155-
token: ${{ steps.app-token.outputs.token }}
156-
path: graphing-library-docs
157-
158-
- name: Trigger downstream doc build
159-
if: github.ref_name == 'migrate-to-gh-actions' && github.event_name == 'push'
160-
run: |
161-
cd graphing-library-docs
162-
git commit --allow-empty -m "deploying https://github.com/plotly/plotly.py/commit/${{ github.sha }}"
163-
git push

0 commit comments

Comments
 (0)