Skip to content

fix: configure GitHub Pages to use Actions workflow source#113

Merged
eurunuela merged 2 commits intoME-ICA:masterfrom
eurunuela:fix/github-pages-workflow-source-clean
Mar 3, 2026
Merged

fix: configure GitHub Pages to use Actions workflow source#113
eurunuela merged 2 commits intoME-ICA:masterfrom
eurunuela:fix/github-pages-workflow-source-clean

Conversation

@eurunuela
Copy link
Collaborator

@eurunuela eurunuela commented Mar 3, 2026

Summary

  • Adds actions/configure-pages@v4 to the build job so the workflow self-configures the Pages source on every non-PR run
  • Prevents silent regression if the UI setting is ever switched back to legacy mode

Root cause

GitHub Pages was set to legacy mode (build_type: legacy), serving raw Markdown via Jekyll from docs/. The actions/deploy-pages@v4 step was deploying the correct MkDocs/Material site, but the legacy build took precedence — causing 404s on all sub-pages.

The immediate fix (switching build_type from legacy to workflow via the GitHub API) has already been applied to the repo settings.

Test plan

🤖 Generated with Claude Code

Add `actions/configure-pages@v4` step to the build job so the workflow
self-configures the Pages source on every non-PR run. This prevents
silent regression if someone switches Pages settings back to legacy mode
in the UI.

The immediate fix (switching build_type from legacy to workflow via the
GitHub API) has already been applied to the repo settings.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 3, 2026 17:03
@netlify
Copy link

netlify bot commented Mar 3, 2026

Deploy Preview for rica-fmri ready!

Name Link
🔨 Latest commit d99afa4
🔍 Latest deploy log https://app.netlify.com/projects/rica-fmri/deploys/69a71b33ca200e00088f5cc5
😎 Deploy Preview https://deploy-preview-113--rica-fmri.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a comment

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 GitHub Actions workflow to ensure GitHub Pages is configured to use the workflow-based deployment source (vs legacy), preventing regressions where legacy Pages builds take precedence over the deployed MkDocs site.

Changes:

  • Adds actions/configure-pages@v4 to the docs workflow build job on non-PR runs.

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

Per review feedback, configure-pages only needs to run when deploying.
Moving it to the deploy job (which already has the Pages write/OIDC
permissions and the non-PR guard) follows least-privilege and avoids
configuring Pages on runs where deployment is skipped.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@eurunuela eurunuela merged commit c1a8959 into ME-ICA:master Mar 3, 2026
1 check passed
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