Sync main to beta after #39#42
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>
The dev channel fails to compile due to the TemplateSelect .state removal on ESPHome dev; excluded until the firmware YAML is updated. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Consolidate workflows into shared ApolloAutomation/Workflows reusable workflows
Sync beta to main: WizMote remote control + IR receiver removal
WalkthroughThis PR updates the dependabot schedule interval and migrates three GitHub Actions workflows (autoassign, ci, weekly) to call reusable workflows from ApolloAutomation/Workflows, adjusting triggers, permissions, and ESPHome build version matrices (excluding ChangesCI/CD Workflow Migration
Estimated code review effort: 2 (Simple) | ~10 minutes Sequence Diagram(s)sequenceDiagram
participant GitHub
participant AutoassignWorkflow as autoassign.yml
participant ReusableAutoassign as Workflows/autoassign.yml
GitHub->>AutoassignWorkflow: issue or PR opened
AutoassignWorkflow->>AutoassignWorkflow: verify not a fork PR
AutoassignWorkflow->>ReusableAutoassign: call with assignees, num-of-assignees
ReusableAutoassign-->>GitHub: assign issue/PR
Possibly related PRs
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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.github/workflows/autoassign.yml (1)
17-20: 🔒 Security & Privacy | 🔵 Trivial | ⚡ Quick winPin the reusable workflows to immutable refs.
.github/workflows/autoassign.yml,.github/workflows/ci.yml, and.github/workflows/weekly.ymlall callApolloAutomation/Workflowsat@main; pinning those to a tag or commit SHA will make CI/CD reproducible and auditable. The reusable workflow only declares inputs (assignees,num-of-assignees), so nosecrets:stanza is needed here.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.github/workflows/autoassign.yml around lines 17 - 20, The reusable workflow invocation in the autoassign workflow is pinned to a moving branch ref, which should be changed to an immutable tag or commit SHA for reproducible and auditable runs. Update the workflow call that uses ApolloAutomation/Workflows/.github/workflows/autoassign.yml in the autoassign workflow to reference a fixed version, and keep only the declared inputs (assignees and num-of-assignees) since no secrets stanza is required.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.github/workflows/autoassign.yml:
- Around line 17-20: The reusable workflow invocation in the autoassign workflow
is pinned to a moving branch ref, which should be changed to an immutable tag or
commit SHA for reproducible and auditable runs. Update the workflow call that
uses ApolloAutomation/Workflows/.github/workflows/autoassign.yml in the
autoassign workflow to reference a fixed version, and keep only the declared
inputs (assignees and num-of-assignees) since no secrets stanza is required.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 465e21f1-b7a5-4554-b7b5-b2ca74dde28c
📒 Files selected for processing (4)
.github/dependabot.yml.github/workflows/autoassign.yml.github/workflows/ci.yml.github/workflows/weekly.yml
Version: 26.7.7.1 (unchanged - workflow files only)
What does this implement/fix?
Syncs main back into beta after #39 merged. Brings the workflow consolidation from #40 (which went directly to main) into beta so the branches don't drift:
No firmware changes.
Types of changes
Checklist / Checklijst:
If user-visible functionality or configuration variables are added/modified:
🤖 Generated with Claude Code
Summary by CodeRabbit