ci: build and deploy docs only on push to main - #322
Merged
Conversation
Contributor
Author
|
Updated: docs are now built on every PR push (no artifact upload) to catch build errors early, while build + deploy to GitHub Pages still runs only on push to |
Contributor
Author
|
Refactored: the deploy step now lives inside the main |
Contributor
Author
|
Reverted to the two-job implementation: docs are built on every PR in |
fsamier
added a commit
that referenced
this pull request
Aug 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes the
deploy docsjob that was timing out onmain(the Pages deployment stayed stuck indeployment_queuedand eventually failed after 10 minutes).Changes
cijob) to catch build errors early, without uploading any artifact.deploy-docsa self-contained job that builds and deploys docs, gated ongithub.event_name == 'push' && github.ref == 'refs/heads/main'.github-pagesenvironment on the deploy job, whichactions/deploy-pagesrequires. Without it the deployment never resolves and times out — the root cause of the failure.Result
Docs are built on every PR to anticipate errors, and built + deployed to GitHub Pages only on push to
main, where the Pages deploy step now resolves correctly instead of hanging.