Skip to content

Commit 35ad65b

Browse files
igerberclaude
andcommitted
Address PR #413 R1 P2: include .readthedocs.yaml in workflow path filters
Reviewer caught that the new sphinx-build job mirrors .readthedocs.yaml (Python version, pandoc apt package, post_install docs deps) but the workflow's path filters omitted .readthedocs.yaml. A future PR changing only RTD config (e.g., Python version bump or new apt package) would skip this docs-build guard, allowing drift between CI's mirror and what RTD actually does. Added '.readthedocs.yaml' to both push.paths and pull_request.paths. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent f8766c3 commit 35ad65b

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

.github/workflows/docs-tests.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ on:
1313
# file itself.
1414
- 'tests/conftest.py'
1515
- 'pyproject.toml'
16+
# sphinx-build job mirrors RTD setup; trigger when RTD config drifts
17+
# (Python version, apt packages, post_install docs deps).
18+
- '.readthedocs.yaml'
1619
- '.github/workflows/docs-tests.yml'
1720
pull_request:
1821
branches: [main]
@@ -23,6 +26,9 @@ on:
2326
- 'tests/test_doc_snippets.py'
2427
- 'tests/conftest.py'
2528
- 'pyproject.toml'
29+
# sphinx-build job mirrors RTD setup; trigger when RTD config drifts
30+
# (Python version, apt packages, post_install docs deps).
31+
- '.readthedocs.yaml'
2632
- '.github/workflows/docs-tests.yml'
2733
schedule:
2834
# Weekly Sunday 6am UTC - smoke test that snippets still execute

0 commit comments

Comments
 (0)