Skip to content

Commit 2000007

Browse files
fix: Remove docs references to test outputs (#267)
The docs workflow currently requires the output of the tests workflow, which is not listed in `docs.needs`. This PR removes all references to test outputs from the docs workflow.
2 parents a7b863a + 227d974 commit 2000007

File tree

3 files changed

+3
-11
lines changed

3 files changed

+3
-11
lines changed

.github/workflows/_docs.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
on:
22
workflow_call:
3-
inputs:
4-
publish:
5-
type: boolean
6-
description: If true, publishes docs to gh-pages
73

84

95
jobs:
@@ -49,7 +45,7 @@ jobs:
4945
run: python .github/pages/make_switcher.py --add $DOCS_VERSION ${{ github.repository }} .github/pages/switcher.json
5046

5147
- name: Publish Docs to gh-pages
52-
if: inputs.publish && (github.ref_type == 'tag' || github.ref_name == 'main')
48+
if: github.ref_type == 'tag' || github.ref_name == 'main'
5349
# We pin to the SHA, not the tag, for security reasons.
5450
# https://docs.github.com/en/actions/learn-github-actions/security-hardening-for-github-actions#using-third-party-actions
5551
uses: peaceiris/actions-gh-pages@4f9cc6602d3f66b9c108549d475ec49e8ef4d45e # v4.0.0

.github/workflows/ci.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@ jobs:
2424

2525
docs:
2626
uses: ./.github/workflows/_docs.yml
27-
with:
28-
publish: ${{needs.test.result == 'success'}}
2927
permissions:
3028
contents: write
3129

template/{% if git_platform=="github.com" %}.github{% endif %}/workflows/ci.yml.jinja

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,11 @@ jobs:
3737
permissions:
3838
contents: read
3939
packages: write
40-
{% endraw %}{% endif %}{% if sphinx %}{% raw %}
40+
{% endraw %}{% endif %}{% if sphinx %}
4141
docs:
4242
uses: ./.github/workflows/_docs.yml
43-
with:
44-
publish: ${{ needs.test.result == 'success' }}
4543

46-
{% endraw %}{% endif %}
44+
{% endif %}
4745
dist:
4846
uses: ./.github/workflows/_dist.yml
4947
{% if pypi %}

0 commit comments

Comments
 (0)