Skip to content

Consolidate workflows into shared ApolloAutomation/Workflows reusable workflows#106

Merged
TrevorSchirmer merged 1 commit into
mainfrom
chore/consolidate-workflows
Jul 7, 2026
Merged

Consolidate workflows into shared ApolloAutomation/Workflows reusable workflows#106
TrevorSchirmer merged 1 commit into
mainfrom
chore/consolidate-workflows

Conversation

@TrevorSchirmer

@TrevorSchirmer TrevorSchirmer commented Jul 7, 2026

Copy link
Copy Markdown
Member

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 pattern build.yml already 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 passes version: through, so the matrix now genuinely tests all three channels.

This PR validates itself: the checks below run the new thin ci.yml → shared esphome-ci.yml matrix.

Types of changes

  • Bugfix (fixed change that fixes an issue)
  • New feature (thanks!)
  • Breaking change (repair/feature that breaks existing functionality)
  • Dependency Update - Does not publish
  • Other - Does not publish
  • Website of github readme file update - Does not publish
  • Github workflows - Does not publish

Checklist / Checklijst:

  • The code change has been tested and works locally
  • The code change has not yet been tested

If user-visible functionality or configuration variables are added/modified:

  • Added/updated documentation for the web page

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated automated dependency checks to run less frequently.
    • Streamlined issue/PR assignment and CI jobs to use shared automation, helping keep checks more consistent.
    • Tightened workflow permissions and added safeguards so automation avoids running on forked pull requests.

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>
@github-actions github-actions Bot added the dependency-update Dependency update label Jul 7, 2026
@github-actions
github-actions Bot requested a review from bharvey88 July 7, 2026 14:16
@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ed40de7f-ca54-4396-8239-08d9ba8a8cd5

📥 Commits

Reviewing files that changed from the base of the PR and between 56853b8 and 2c15c56.

📒 Files selected for processing (4)
  • .github/dependabot.yml
  • .github/workflows/autoassign.yml
  • .github/workflows/ci.yml
  • .github/workflows/weekly.yml

Walkthrough

This 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.

Changes

Reusable Workflow Migration

Layer / File(s) Summary
Dependabot schedule update
.github/dependabot.yml
Schedule interval changed from daily to monthly; comment header replaced.
Auto-assign delegation
.github/workflows/autoassign.yml
Removed inline pozil/auto-assign-issue@v4 step; added workflow-level permissions, non-fork condition check, and delegation to reusable autoassign workflow with num-of-assignees input.
CI build delegation
.github/workflows/ci.yml
Dropped id-token: write permission; label-check and ci jobs now call reusable label-check.yml and esphome-ci.yml workflows with explicit yaml-files list, removing the prior matrix-based esphome/build-action build.
Weekly build delegation
.github/workflows/weekly.yml
Set contents: read permission; replaced matrix-based build job with a call to reusable esphome-ci.yml workflow passing a yaml-files list.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested labels: github_actions

Poem

A rabbit hops through YAML fields so neat,
Trading daily hops for a monthly beat 🐇
Workflows now call friends afar,
Reusable helpers, near and far,
Fork PRs skipped, assignees set right—
Hop, hop, hooray, the CI's tight! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: consolidating repository workflows into shared reusable workflows.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/consolidate-workflows

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@TrevorSchirmer
TrevorSchirmer merged commit 12a85fe into main Jul 7, 2026
13 checks passed
@TrevorSchirmer
TrevorSchirmer deleted the chore/consolidate-workflows branch July 7, 2026 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependency-update Dependency update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants