Skip to content

Fix PR and main pages deploys#6

Merged
PhilipMathieu merged 2 commits intomainfrom
claude/fix-pr-preview-deploy-01BHa8uuPpeKWwLCjzYZWwUk
Nov 16, 2025
Merged

Fix PR and main pages deploys#6
PhilipMathieu merged 2 commits intomainfrom
claude/fix-pr-preview-deploy-01BHa8uuPpeKWwLCjzYZWwUk

Conversation

@PhilipMathieu
Copy link
Owner

No description provided.

The previous setup used artifact-based GitHub Pages deployment which
conflicted with the PR preview action that pushes to the gh-pages branch.
This change switches the main deployment to also use the gh-pages branch,
making both workflows compatible.

Changes:
- Replace artifact-based deployment with peaceiris/actions-gh-pages action
- Configure to deploy from gh-pages branch
- Add keep_files: true to preserve PR preview directories
- Simplify permissions to only require contents: write

Note: Repository Pages settings must be configured to deploy from the
gh-pages branch for this to work.
Simplified deployment by merging static.yml and pr-preview.yml into a
single pages.yml workflow. The rossjrw/pr-preview-action handles both:
- Main branch pushes → deploy to gh-pages root (production)
- Pull requests → deploy to gh-pages/pr-previews/pr-{number}
- PR close → cleanup preview directory

This reduces complexity and maintenance while providing the same
functionality. One workflow, one action, all deployments.

Changes:
- Merged pr-preview.yml and static.yml into pages.yml
- Added push trigger for main branch deployment
- Renamed jobs to 'deploy' and 'cleanup' for clarity
- Removed redundant static.yml workflow
@PhilipMathieu PhilipMathieu merged commit d511d14 into main Nov 16, 2025
10 of 14 checks passed
@PhilipMathieu PhilipMathieu deleted the claude/fix-pr-preview-deploy-01BHa8uuPpeKWwLCjzYZWwUk branch November 16, 2025 22:00
@github-actions
Copy link

github-actions bot commented Nov 16, 2025

PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2025-11-16 22:00 UTC

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