Skip to content

Document sphinx-multiversion workflow trigger requirements#15

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/build-sphinx-multiversion
Draft

Document sphinx-multiversion workflow trigger requirements#15
Copilot wants to merge 2 commits intomainfrom
copilot/build-sphinx-multiversion

Conversation

Copy link
Contributor

Copilot AI commented Feb 9, 2026

sphinx-multiversion builds all documentation versions (tags + branches) in a single pass by scanning git history. Both workflow triggers are required:

  • Tag trigger: Rebuilds full docs when new versions are tagged
  • Main branch trigger: Rebuilds full docs when development version changes
  • fetch-depth: 0: Ensures full history access for version discovery

Changes

  • .github/workflows/docs.yml: Added inline comments explaining trigger and fetch-depth requirements
  • docs/README.md: Added comprehensive documentation on multiversion setup, why both triggers are essential, and local build instructions
  • docs/source/conf.py: Added cross-reference comments to workflow documentation

Context

The workflow configuration was already correct but lacked documentation explaining why both triggers are necessary. Removing either trigger would result in incomplete or stale documentation builds, as sphinx-multiversion would only see partial git history.

Original prompt

Do i need to trigger on tag or on main branch only because im using sphinx-multiversion for building for multiple tags


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…y comments

Co-authored-by: digreatbrian <119015367+digreatbrian@users.noreply.github.com>
Copilot AI changed the title [WIP] Determine trigger source for Sphinx multiversion builds Document sphinx-multiversion workflow trigger requirements Feb 9, 2026
Copilot AI requested a review from digreatbrian February 9, 2026 17:50
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.

2 participants