Skip to content

Publish repo root to gh-pages again (restore publish_dir) - #14

Merged
james merged 1 commit into
mainfrom
fix/serialise-gh-pages-deploys
Oct 12, 2026
Merged

james merged 1 commit into
mainfrom
fix/serialise-gh-pages-deploys

Conversation

@james

@james james commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator

Deploys from main haven't actually done anything since c2f2755. That commit replaced publish_dir: ., exclude_assets and enable_jekyll with keep_files: true and left publish_dir unset. peaceiris/actions-gh-pages then defaults to ./public, which doesn't exist in this repo, so the job reports success without publishing anything.

Deploys from main have been no-ops since c2f2755 ("Don't overwrite
pr-preview in github pages"). That commit replaced `publish_dir: .`,
`exclude_assets` and `enable_jekyll` with `keep_files: true` and left
publish_dir unset. peaceiris/actions-gh-pages then defaults to
`./public`, which doesn't exist in this repo, so the job reports
success without publishing anything.

Result: merges to main (e.g. PR #12) never reached the live site; only
PR previews did.

Restore `publish_dir: .`, keeping `keep_files: true` so pr-preview/ is
preserved.

Broken by: c2f2755
@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2026-10-12 08:07 UTC

@james

james commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator Author

Sorry about the repeated reviews @jabley , only the merge to main is broken

@james
james requested a review from jabley October 9, 2026 17:04
@james
james merged commit 9cfc467 into main Oct 12, 2026
2 checks 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