Skip to content

Detect doc build failure - #10

Merged
glesur merged 4 commits into
masterfrom
docFailureDetection
Jun 25, 2026
Merged

Detect doc build failure#10
glesur merged 4 commits into
masterfrom
docFailureDetection

Conversation

@glesur

@glesur glesur commented Jun 25, 2026

Copy link
Copy Markdown
Owner

No description provided.

@glesur glesur added the documentation Improvements or additions to documentation label Jun 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the documentation build workflow to run on pull requests (so doc build failures are caught earlier) and tightens the docs to avoid formatting/warning issues during Sphinx builds.

Changes:

  • Run the documentation build workflow on pull_request events targeting master.
  • Treat Sphinx warnings as errors (intended via -W) and prevent Pages deployment on non-push events.
  • Minor reStructuredText table formatting/whitespace cleanup in the docs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
doc/source/input_file.rst Adjusts table formatting/whitespace in docs (likely to keep Sphinx output clean).
.github/workflows/astra-ci-doc.yml Adds PR trigger and deploy gating; attempts to enforce “warnings as errors” for doc builds.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/astra-ci-doc.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@glesur
glesur merged commit ba4f157 into master Jun 25, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants