Skip to content

Deploy the built website to GitHub Pages instead of the README - #5

Merged
charles2ke merged 1 commit into
mainfrom
copilot/deploy-website-to-pages
Aug 27, 2026
Merged

Deploy the built website to GitHub Pages instead of the README#5
charles2ke merged 1 commit into
mainfrom
copilot/deploy-website-to-pages

Conversation

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Summary

GitHub Pages was still using the legacy "Deploy from a branch" source, so the Jekyll-rendered README.md was published instead of the built website. The Actions deploy job also failed with 404 ... Ensure GitHub Pages has been enabled.

Changes

  • Added an actions/configure-pages@v5 step (with enablement: true) to .github/workflows/deploy-pages.yml so Pages is enabled and its source is switched to GitHub Actions, publishing website/dist instead of the README.

Validation

  • npm ci + GITHUB_PAGES=true npm run build in website/ succeeds and produces dist/index.html with correct /GraphQL/ base paths.
  • CodeQL (actions): 0 alerts.

Note: if the workflow token cannot change the Pages setting, set Settings → Pages → Source to "GitHub Actions" once and re-run the workflow.

Co-authored-by: charles2ke <6725706+charles2ke@users.noreply.github.com>
@charles2ke
charles2ke marked this pull request as ready for review August 27, 2026 10:06
@charles2ke
charles2ke requested a lite review from Copilot August 27, 2026 10:06
@charles2ke
charles2ke merged commit 6ce3f58 into main Aug 27, 2026
2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The added configure-pages enablement step may fail under the default github.token permissions (blocking deploy) unless an appropriate token/strategy is provided.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR updates the GitHub Pages deployment workflow so the repository publishes the built static site (website/dist) via GitHub Actions, rather than publishing the Jekyll-rendered README.md from the legacy “Deploy from a branch” Pages source.

Changes:

  • Adds a actions/configure-pages@v5 step with enablement: true to enable Pages and switch the Pages source to GitHub Actions before building/deploying.
  • Keeps the existing build (npm ci + GITHUB_PAGES=true npm run build) and deploy (upload-pages-artifact + deploy-pages) flow.
File summaries
File Description
.github/workflows/deploy-pages.yml Adds a Pages configuration step intended to enable GitHub Pages and switch the source to GitHub Actions before deploying the built site artifact.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +28 to +34
- name: Configure GitHub Pages
uses: actions/configure-pages@v5
with:
# Enables Pages and switches the source to GitHub Actions so the built
# website is published instead of the Jekyll-rendered README.
enablement: true

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.

3 participants