Skip to content

Comments

Put copies of gh-aw workflows in .github instead of symlinks#32

Merged
strawgate merged 4 commits intomainfrom
use-workflow-copies
Feb 15, 2026
Merged

Put copies of gh-aw workflows in .github instead of symlinks#32
strawgate merged 4 commits intomainfrom
use-workflow-copies

Conversation

@strawgate
Copy link
Collaborator

No description provided.

@echo " lint-actions - Validate GitHub Actions composite action files"
@echo " sync - Ensure symlinks are real (core.symlinks=false fix)"
@echo " compile - Ensure symlinks + compile agentic workflows to lock files"
@echo " sync - Run scripts/dogfood.sh to copy shims, prompts, and fragments"
Copy link
Contributor

Choose a reason for hiding this comment

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

⚪ [LOW] Help text is inaccurate — script only copies shims

The help string says "copy shims, prompts, and fragments" but scripts/dogfood.sh now only copies shims. Prompts and fragments are no longer copied (they live directly in .github/workflows/).

Suggested change
@echo " sync - Run scripts/dogfood.sh to copy shims, prompts, and fragments"
`@echo` " sync - Run scripts/dogfood.sh to copy shims into .github/workflows/"

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Reviewed all changed files. The implementation correctly replaces symlinks with synced copies, creates an automated sync script (scripts/dogfood.sh), and updates documentation consistently. Lock files were recompiled as expected. No new issues found beyond the existing unresolved thread about the Makefile help text.

Generated by Pr Review for issue #32

@strawgate strawgate merged commit 2c522b9 into main Feb 15, 2026
9 checks passed
@strawgate strawgate deleted the use-workflow-copies branch February 15, 2026 23:59
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.

1 participant