Consolidate workflows into shared ApolloAutomation/Workflows reusable workflows#106
Conversation
Replaces inline version-pinned actions with thin callers of ApolloAutomation/Workflows reusable workflows referenced @main. Leaf repo now contains zero dependabot-bumpable refs; action version pins are centralized in the Workflows repo. Dependabot schedule relaxed to monthly as a safety net. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
WalkthroughThis PR migrates CI/CD workflows to use reusable workflows from ApolloAutomation/Workflows. The dependabot schedule changes from daily to monthly. Autoassign, CI, and weekly build workflows are rewritten to delegate to shared reusable workflows instead of inline job logic. ChangesReusable Workflow Migration
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Version:
What does this implement/fix?
Consolidates this repo's CI, weekly build, and auto-assign workflows into thin callers of the shared ApolloAutomation/Workflows reusable workflows (
esphome-ci.yml,autoassign.yml), the same patternbuild.ymlalready uses. The repo now contains zero version-pinned actions, so Dependabot has nothing to bump here — action updates land as a single grouped PR in the Workflows repo and propagate to the whole fleet via@main. The local dependabot schedule is relaxed to monthly as a safety net.Also fixes a latent CI bug: the old inline matrix listed ESPHome stable/beta/dev but never passed the version to
esphome/build-action, so all three legs built the same default version. The shared workflow passesversion:through, so the matrix now genuinely tests all three channels.This PR validates itself: the checks below run the new thin
ci.yml→ sharedesphome-ci.ymlmatrix.Types of changes
Checklist / Checklijst:
If user-visible functionality or configuration variables are added/modified:
🤖 Generated with Claude Code
Summary by CodeRabbit