Skip to content

Re-enable the dev leg of the ESPHome CI matrix - #82

Merged
bharvey88 merged 3 commits into
betafrom
ci/enable-dev-leg
Sep 19, 2026
Merged

bharvey88 merged 3 commits into
betafrom
ci/enable-dev-leg

Conversation

@bharvey88

@bharvey88 bharvey88 commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

What does this implement/fix?

Adds dev back to the ESPHome CI matrix.

It was dropped on 2026-07-07 in 1d5470b "until the TemplateSelect .state incompatibility is fixed". That was fixed the same day in 2d2d311, and again for the wizmote selects on 2026-07-12 in 8a5ed17, but the leg was never turned back on.

With it off, two breaks against ESPHome dev went unnoticed until a local dev build caught them: the OTA manifest lambda (#80) and the esp32_improv deprecation (#81).

Targets beta: ci.yml runs on pull_request, so beta's copy is the one that gates firmware PRs. The next release carries it to main, keeping the two in sync.

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

TrevorSchirmer and others added 2 commits August 28, 2026 17:01
Release 26.8.28.1 - Launch firmware: Music Assistant playback fix, channel_colors migration
The dev leg was dropped on 2026-07-07 (1d5470b) with the note "dev
excluded until the TemplateSelect .state incompatibility is fixed". That
incompatibility was fixed the same day in 2d2d311, and again for the
wizmote selects on 2026-07-12 in 8a5ed17. Every select in the config now
uses current_option(); the remaining .state calls are on text, switch and
binary_sensor components, where it is still correct.

The leg was never turned back on, so two breaks against ESPHome dev went
unnoticed until a local dev build surfaced them: the OTA manifest lambda
no longer compiling against the new set_source_url signature, and the
esp32_improv deprecation. The first would have broken builds for anyone
compiling this YAML on the day 2026.10.0 released.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 45 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: f3789e21-adc9-40b9-9565-bdc9a8e94f4d

📥 Commits

Reviewing files that changed from the base of the PR and between 991728d and e26ecaf.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

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.

Retargeting this PR from main to beta so main and beta stay in sync;
the next release carries it to main.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@bharvey88
bharvey88 changed the base branch from main to beta September 18, 2026 23:10
@bharvey88
bharvey88 marked this pull request as ready for review September 18, 2026 23:10
@bharvey88
bharvey88 merged commit 299c475 into beta Sep 19, 2026
8 checks passed
@bharvey88
bharvey88 deleted the ci/enable-dev-leg branch September 19, 2026 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants