-
由 Will Chen 提交于
## Summary - Adds a `check-changes` job to CI that detects if a PR only modifies files in `.claude/` - Skips the expensive `test` and `merge-reports` jobs when only Claude config files are changed - Always runs tests on pushes to main branch for safety ## Test plan - Create a PR that only changes files in `.claude/` → CI should skip E2E tests - Create a PR that changes files outside `.claude/` → CI should run E2E tests normally - Push to main → CI should always run E2E tests #skip-bugbot
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2362"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Skip E2E tests in CI when a PR only changes files in .claude/, cutting build time and saving resources. Tests always run on pushes to main; aligns with Linear issue 1769548046265. - **New Features** - Added a check-changes job that outputs should_run_tests based on the diff (PR vs push). - Gated test and merge-reports jobs on should_run_tests; skip when only .claude/ files changed. - Falls back to running tests if changed files can’t be determined. <sup>Written for commit a75bfed9498b31634a8944d63a435688e0bac8ac. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Claude Opus 4.5 <noreply@anthropic.com>