Skip to content

Deploy validated main builds to GitHub Pages - #306

Merged
alexthemitchell merged 3 commits into
mainfrom
copilot/setup-github-pages-deployment
Aug 29, 2026
Merged

Deploy validated main builds to GitHub Pages#306
alexthemitchell merged 3 commits into
mainfrom
copilot/setup-github-pages-deployment

Conversation

Copilot AI commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

GitHub Pages is introduced as the first deployment target. Deployments promote CI’s validated production artifact rather than rebuilding application assets.

  • Deployment workflow
    • Triggers after successful CI push runs on main
    • Downloads the matching rad-io-dist-<SHA> artifact
    • Uploads it as a Pages artifact and deploys through the github-pages environment
    • Uses scoped Pages and OIDC permissions with serialized deployments
if: github.event.workflow_run.conclusion == 'success' &&
    github.event.workflow_run.event == 'push'
  • Documentation
    • Updates CI documentation to describe GitHub Pages artifact promotion behavior.

Copilot AI lite review requested due to automatic review settings August 28, 2026 23:29

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.

Copilot wasn't able to review any files in this pull request.


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

Copilot AI linked an issue Aug 28, 2026 that may be closed by this pull request
Co-authored-by: alexthemitchell <5687158+alexthemitchell@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 28, 2026 23:31
Copilot AI changed the title [WIP] Add GitHub Pages as deployment target Deploy validated main builds to GitHub Pages Aug 28, 2026
Copilot AI requested a review from alexthemitchell August 28, 2026 23:32

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.

🟡 Changes recommended

The new Pages workflow is missing actions: read token permissions needed to download artifacts from a different workflow run via run-id.

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

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread .github/workflows/pages.yml
@alexthemitchell
alexthemitchell marked this pull request as ready for review August 29, 2026 02:05
@alexthemitchell

Copy link
Copy Markdown
Owner

@copilot Fix the code for all comments in this review thread.

When a review comment includes a suggested change, apply the suggestion exactly.

Do not make changes beyond what is described in the linked review thread.

Co-authored-by: alexthemitchell <5687158+alexthemitchell@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 29, 2026 02:07

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI commented Aug 29, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Fix the code for all comments in this review thread.

When a review comment includes a suggested change, app...

Added actions: read to the Pages workflow permissions in aa198ac.

@alexthemitchell
alexthemitchell merged commit 07e84fa into main Aug 29, 2026
6 checks passed
@alexthemitchell
alexthemitchell deleted the copilot/setup-github-pages-deployment branch August 29, 2026 02:23
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.

Setup GitHub Pages as first deployment target

3 participants