ci: add required ci.yml workflow - #48
Conversation
Adds the required ci.yml workflow that delegates to the org-level reusable workflow (ci-reusable.yml@main), following the same pattern as claude.yml. Closes #20 Co-authored-by: don-petry <don-petry@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds a GitHub Actions workflow file Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related issues
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@don-petry — no CODEOWNERS file found, so tagging you directly. This PR adds the required |
|
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds a required GitHub Actions CI entrypoint workflow that delegates execution to the org-level reusable CI workflow, addressing the missing-ci.yml compliance finding (closes #20).
Changes:
- Introduces
.github/workflows/ci.ymlas a thin caller workflow for CI onpush/pull_requesttomain - Delegates all CI logic to
petry-projects/.github/.github/workflows/ci-reusable.yml@main - Sets job-level token permissions for the reusable workflow run
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| jobs: | ||
| ci: | ||
| uses: petry-projects/.github/.github/workflows/ci-reusable.yml@main |
There was a problem hiding this comment.
Calling a reusable workflow by a floating ref (@main) creates supply-chain risk and can cause unexpected behavior changes over time. Prefer pinning to an immutable ref (tag or full commit SHA) and periodically updating it intentionally.
| uses: petry-projects/.github/.github/workflows/ci-reusable.yml@main | |
| uses: petry-projects/.github/.github/workflows/ci-reusable.yml@<FULL_COMMIT_SHA> |
| jobs: | ||
| ci: | ||
| uses: petry-projects/.github/.github/workflows/ci-reusable.yml@main | ||
| secrets: inherit |
There was a problem hiding this comment.
Using secrets: inherit passes all repository/environment secrets to the reusable workflow, which is broader than necessary for many CI runs. Consider explicitly mapping only the required secrets to reduce blast radius if the called workflow changes or is misused.
| secrets: inherit | |
| secrets: {} |
|
Closing as stale — predates the standards PR. This PR was generated by Claude during the original bulk-toggle yesterday, before petry-projects/.github#86 landed. That standards PR added prompt rules that:
Re-toggling the underlying issue will let Claude regenerate this fix using the new rules. The next run should produce a workflow that is byte-identical to the standard template (verified with the canary on TalkTerm#51 → PR #78 yesterday). |


Summary
.github/workflows/ci.ymlas a thin caller that delegates to the org-level reusable workflow (ci-reusable.yml@main)claude.yml(and other existing workflows in this repo)missing-ci.ymlfindingCloses #20
Generated with Claude Code
Summary by CodeRabbit