justin/ai-workflow-course
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Drop the bad gh-mirror/ exclude (it blocked the auto-heal) #112

Merged
claude merged 1 commits from fix/drop-bad-exclude into main 2026-06-24 21:43:45 -04:00
Conversation 0 Commits 1 Files Changed 1
+4 -2
claude commented 2026-06-24 21:43:23 -04:00
Contributor
Copy Link
Copy Source

Rsync's --exclude also protects matching paths at the DESTINATION from --delete. The belt-and-suspenders --exclude='gh-mirror/' in the previous fix kept the stray gh-mirror/ subtree alive on GitHub forever. The structural fix (clone into $RUNNER_TEMP) already prevents the recursive self-include, so the exclude is unnecessary and harmful. NOTE comment left in-place so a future maintainer doesn't re-add it.

Merging triggers the next sync, which will finally delete the stray dir from GitHub.

🤖 Generated with Claude Code

Rsync's `--exclude` also protects matching paths at the DESTINATION from `--delete`. The belt-and-suspenders `--exclude='gh-mirror/'` in the previous fix kept the stray `gh-mirror/` subtree alive on GitHub forever. The structural fix (clone into `$RUNNER_TEMP`) already prevents the recursive self-include, so the exclude is unnecessary and harmful. NOTE comment left in-place so a future maintainer doesn't re-add it. Merging triggers the next sync, which will finally delete the stray dir from GitHub. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
claude added 1 commit 2026-06-24 21:43:23 -04:00
fix(ci): drop the bad gh-mirror/ rsync exclude (it blocked --delete cleanup)
CI / check (pull_request) Successful in 6s
Details
5286114acb 
rsync's --exclude also protects the matching path at the DESTINATION from
--delete. The belt-and-suspenders --exclude='gh-mirror/' I added in the previous
fix prevented the stray gh-mirror/ subtree at the destination from ever being
deleted by --delete, defeating the auto-heal. The structural fix (cloning the
GitHub mirror into $RUNNER_TEMP, outside ./) already prevents the recursive
self-include, so the exclude is unnecessary and harmful. Removed it; left an
explanatory NOTE in the workflow so this isn't re-added.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
Claude-Session: https://claude.ai/code/session_01TfzV5QvtPDz8LJS3Pu5VLT
claude merged commit 125802616d into main 2026-06-24 21:43:45 -04:00
claude deleted branch fix/drop-bad-exclude 2026-06-24 21:43:45 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
ai-ready
Scoped and ready to implement
 autopilot
Drive end-to-end autonomously (claude-deck)
 autopilot-careful
Autopilot may scope/implement but never auto-merge
 bug
Defect
 feature
New capability
 needs-human
Autopilot stalled or vetoed — a human must act
 P0
Critical
 P1
High
 P2
Normal
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
justin (Justin Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: justin/ai-workflow-course#112