Skip to content

docs(claude): document Pages source-path requirement (#55) - #56

Merged
jayrav13 merged 2 commits into
mainfrom
worktree-fix-55-document-pages-source
Apr 28, 2026
Merged

docs(claude): document Pages source-path requirement (#55)#56
jayrav13 merged 2 commits into
mainfrom
worktree-fix-55-document-pages-source

Conversation

@jayrav13

Copy link
Copy Markdown
Owner

Summary

Closes #55. Adds a Pages-source subsection to CLAUDE.md documenting the silent-stale-deploy trap we hit while investigating #53.

`mkdocs gh-deploy` writes the built site (with `.nojekyll`) to the root of `gh-pages`. If the GitHub Pages source path in repo settings is set to `/docs` instead of `/`, the separate `pages-build-deployment` workflow fails at `dir_chdir0 - /github/workspace/docs`, and Pages keeps serving the previously-published artifact — so the live site looks merely "stale" rather than broken. The setting has already been corrected; this just documents the requirement and a diagnostic flow so the next contributor (or future-Claude) doesn't re-investigate it.

No code changes — single `CLAUDE.md` edit, +33 lines.

Test plan

  • Pre-push hooks (lint + pytest) passed locally — 434 tests
  • CI green on this PR

Co-authored by Claude

jayrav13 and others added 2 commits April 27, 2026 23:54
Captures the silent-stale-deploy trap discovered while investigating #53:
mkdocs gh-deploy writes to gh-pages root, but the Pages source-path
setting in repo settings was set to /docs. The pages-build-deployment
workflow fails at dir_chdir0 - /github/workspace/docs and Pages keeps
serving the previous artifact, so the live site looks merely "stale"
instead of broken. Adds a Pages-source subsection under "Docs site" with
the correct config, the diagnostic commands to use when a deploy looks
absent, and the API call to repair the setting.

Closes #55

Co-Authored-By: Claude <noreply@anthropic.com>
The version-bump CI check requires every PR to add a CHANGELOG entry. Documents the same change as the CLAUDE.md update.

Refs #55

Co-Authored-By: Claude <noreply@anthropic.com>
@jayrav13
jayrav13 merged commit 7781f1b into main Apr 28, 2026
6 checks passed
@jayrav13
jayrav13 deleted the worktree-fix-55-document-pages-source branch April 28, 2026 04:03
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.

docs(claude): document Pages source-path requirement after #53 deploy investigation

1 participant