Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/workflows/docs-sync.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ on:
- main
paths:
- "docs/**"
# Manual trigger. Needed because the push trigger only fires on docs/** changes,
# so a workflow-only fix cannot re-run the sync, and a sync PR left stale by a
# failed run has no way to catch up until the next docs edit.
workflow_dispatch:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Manual dispatch uses selected branch

When an operator dispatches this workflow from a non-main branch, the unpinned source checkout transforms that branch's documentation and commits it to a docs-site PR targeting main, causing stale or feature-branch content to enter the sync PR; an older branch without the transform script instead makes the run fail.

Comment on lines +14 to +17

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Manual trigger lacks changelog entry

This workflow change has no corresponding entry in the Unreleased changelog, despite the repository requirement to document every change, so the release history omits the newly supported manual docs-sync operation.

Context Used: CLAUDE.md (source)

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!


# Serialize runs. Two overlapping runs would each check out the same revision of
# the open sync branch, commit divergently, and the loser's push would be
Expand Down