Skip to content

CI: Skip documentation deployment on pull requests#7

Merged
pgit merged 2 commits intomasterfrom
copilot/verify-ci-configuration
Feb 4, 2026
Merged

CI: Skip documentation deployment on pull requests#7
pgit merged 2 commits intomasterfrom
copilot/verify-ci-configuration

Conversation

Copy link
Contributor

Copilot AI commented Feb 4, 2026

The gh-pages.yml workflow was deploying documentation to GitHub Pages on both master pushes and pull requests. PRs should only validate that docs build successfully.

Changes

  • Added conditional to deployment step: if: github.event_name == 'push' && github.ref == 'refs/heads/master'
  • Documentation still builds on PRs (validates compilation), but only deploys on master

Behavior

Pull requests: Build + test + link check + doc build validation
Master branch: Build + test + link check + doc build + deploy to gh-pages


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

Co-authored-by: pgit <1456612+pgit@users.noreply.github.com>
Copilot AI changed the title [WIP] Review CI configuration for documentation deployment CI: Skip documentation deployment on pull requests Feb 4, 2026
Copilot AI requested a review from pgit February 4, 2026 20:07
@pgit pgit marked this pull request as ready for review February 4, 2026 20:40
@pgit pgit merged commit 9683eff into master Feb 4, 2026
4 checks passed
@pgit pgit deleted the copilot/verify-ci-configuration branch February 4, 2026 20:41
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