Skip to content

ci: fix Pages deployment — switch to native artifact model - #8

Merged
rgutzen merged 2 commits into
Lindsay-Lab:mainfrom
rgutzen:chore/fix-pages-deployment
Jul 2, 2026
Merged

ci: fix Pages deployment — switch to native artifact model#8
rgutzen merged 2 commits into
Lindsay-Lab:mainfrom
rgutzen:chore/fix-pages-deployment

Conversation

@rgutzen

@rgutzen rgutzen commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

The peaceiris/actions-gh-pages branch-push approach was correctly pushing MkDocs output to gh-pages, but the repo has a workflow-based deployment using actions/deploy-pages@v5 that was stuck in a deployment_queued loop — the two models conflict.

This switches to the native GitHub Pages artifact model:

  • actions/configure-pages@v5 — sets up the Pages environment
  • actions/upload-pages-artifact@v3 — uploads the MkDocs site/ output
  • actions/deploy-pages@v4 — deploys the artifact

Also switched the Pages source from gh-pages / legacy to workflow mode via the Pages API.

The peaceiris/actions-gh-pages branch-push approach was correctly pushing
MkDocs output to gh-pages, but the repo has a deployed actions/deploy-pages@v5
workflow running as well. This created a 'deployment_queued' loop in the
workflow-based model that never resolved.

Switch to the native GitHub Pages artifact model:
- actions/configure-pages@v5 sets up the Pages environment
- actions/upload-pages-artifact@v3 uploads the MkDocs site/ output
- actions/deploy-pages@v4 deploys the artifact

Also switched the Pages source from 'gh-pages/ legacy' to 'workflow' mode
via the Pages API.
Copilot AI review requested due to automatic review settings July 2, 2026 16:58

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

Updates the documentation deployment workflow to use GitHub Pages’ native artifact-based deployment model instead of pushing to a gh-pages branch, aligning the pipeline with configure-pages + upload-pages-artifact + deploy-pages.

Changes:

  • Switch workflow permissions from contents: write to the Pages/OIDC permission set (contents: read, pages: write, id-token: write).
  • Replace peaceiris/actions-gh-pages branch-push deployment with Pages artifact upload + actions/deploy-pages.
  • Add workflow-level concurrency configuration for Pages-related runs.

Comment thread .github/workflows/docs.yml
Comment on lines +58 to +62
- name: Setup Pages
uses: actions/configure-pages@v5

- name: Upload build artifacts
if: github.event_name == 'pull_request'
uses: actions/upload-artifact@v4
- name: Upload artifact
uses: actions/upload-pages-artifact@v3
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@rgutzen
rgutzen merged commit 591f0b3 into Lindsay-Lab:main Jul 2, 2026
7 checks passed
@rgutzen
rgutzen deleted the chore/fix-pages-deployment branch July 2, 2026 19:22
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