Skip to content

ci: build the docs on PRs via Heart's docs-build.yml (cti-upkeep 3c) - #126

Merged
Jammy2211 merged 1 commit into
mainfrom
feature/cti-docs-yml-caller
Nov 6, 2026
Merged

Jammy2211 merged 1 commit into
mainfrom
feature/cti-docs-yml-caller

Conversation

@Jammy2211

Copy link
Copy Markdown
Collaborator

Summary

PyAutoCTI's Sphinx docs are now built on every PR that touches them, through Heart's reusable docs-build.yml (PyAutoLabs/PyAutoHeart#325). The check fails when the warning count rises above docs/sphinx_warning_baseline.txt. Until now nothing checked the docs before merge, which is how docs/api drifted to 53 dead entries (fixed in #124). Closes #125 (cti-upkeep 3c).

API Changes

None. CI and docs configuration only.

Test Plan

  • Local build under the new conf.py, with arcticpy, warnings written via -w as the workflow counts them: 15 lines, 0 failed imports
  • Same build without arcticpy: 147 lines, with the Trap/ROE pages failing to import. That's why Heart#325 installs arcticpy for autocti.
  • Docs leg green on this PR once Heart#325 is on main (it runs docs-build.yml@main; before that, autocti is an unknown package)
  • Witness: a throwaway branch with a broken autosummary entry turns the Docs leg red (no PR; branch deleted)
Full API Changes (for automation & release notes)

Added

  • .github/workflows/docs.yml: thin caller (package: autocti), same triggers as PyAutoFit's.
  • docs/sphinx_warning_baseline.txt: 15.

Changed

  • docs/conf.py: PyAutoLens-style path setup. It no longer clones PyAutoFit/PyAutoArray from rhayes777/Jammy2211 at build time; that needed pyprojroot and would shadow the branch-matched dependencies CI installs.

Removed

  • docs/requirements.txt: Read the Docs installs .[docs] per .readthedocs.yaml, no sibling keeps one, and it pinned autonerves==2024.11.6.1.

Read the Docs: .readthedocs.yaml (.[docs] + arcticpy post_install) is unchanged. The RTD build on main gets checked after merge.

Tier: glance. Merge mode: auto-merge on green if the Witness passes. Heart STALE at ship time, human-acknowledged.

🤖 Generated with Claude Code

… cloning in conf.py

- .github/workflows/docs.yml: thin caller (package: autocti), mirroring PyAutoFit's
- docs/conf.py: PyAutoLens-style path setup; no more cloning PyAutoFit/PyAutoArray
  from the old personal-account URLs at build time (needed pyprojroot, and would
  shadow the branch-matched deps CI installs)
- docs/requirements.txt removed: Read the Docs installs .[docs] (.readthedocs.yaml)
  and no sibling keeps one; it pinned autonerves==2024.11.6.1
- docs/sphinx_warning_baseline.txt: 15 (local build with arcticpy; re-set from CI if it differs)

Closes #125

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Jammy2211
Jammy2211 merged commit 08040ff into main Nov 6, 2026
4 of 5 checks passed
@Jammy2211
Jammy2211 deleted the feature/cti-docs-yml-caller branch November 6, 2026 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: build the docs on PRs via Heart's docs-build.yml (cti-upkeep 3c)

1 participant