Skip to content

Commit ed8a8a0

Browse files
authored
fix: docs sync PR step — stage and commit review items before creating PR (CopilotKit#3779)
## Summary - Fixes the \"Create PR for review items\" step in the docs sync workflow - The step was creating a branch but never staging/committing the review-item files before calling `gh pr create`, causing `No commits between main and docs-sync-review/...` - Now stages remaining content changes, commits them, pushes the branch, then creates the PR - If no review items are actually changed, exits cleanly instead of failing ## Test plan - [ ] Trigger docs sync workflow manually and verify it handles both `auto_push` and `push_and_pr` paths 🤖 Generated with [Claude Code](https://claude.com/claude-code)
2 parents 629e042 + c3bb61c commit ed8a8a0

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

.github/workflows/showcase_docs-sync.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,15 @@ jobs:
9898
9999
git checkout -b "$BRANCH"
100100
101+
# Stage any remaining unstaged changes (review items the sync script flagged)
102+
git add showcase/shell/src/content/
103+
if git diff --cached --quiet; then
104+
echo "No review items to commit — skipping PR"
105+
exit 0
106+
fi
107+
git commit -m "Docs sync: files needing review (${SHORT_SHA})"
108+
git push origin "$BRANCH"
109+
101110
# The review-items.txt was written by the sync script
102111
REVIEW_ITEMS=$(cat review-items.txt 2>/dev/null || echo "See sync output for details")
103112

0 commit comments

Comments
 (0)