Skip to content

Replace deprecated rgb_order with channel_colors - #70

Merged
bharvey88 merged 3 commits into
betafrom
channel-colors-migration
Aug 30, 2026
Merged

Replace deprecated rgb_order with channel_colors#70
bharvey88 merged 3 commits into
betafrom
channel-colors-migration

Conversation

@TrevorSchirmer

@TrevorSchirmer TrevorSchirmer commented Aug 28, 2026

Copy link
Copy Markdown
Member

ESPHome 2026.8.0 replaces rgb_order / is_rgbw / is_wrgb with a single channel_colors key on esp32_rmt_led_strip (esphome#18474).

channel_colors takes each of R, G, B exactly once, optionally with a single W; the value is case-insensitive. No behaviour change — grb and GRB are equivalent, uppercase to match the ESPHome docs.

Summary by CodeRabbit

  • Compatibility
    • Updated ESPHome device configurations to require ESPHome 2026.8.0 or later.
    • Updated Apollo firmware version references across supported configurations.
  • Bug Fixes
    • Updated RGB LED color-channel configuration for improved compatibility with current ESPHome settings.
    • Applied the LED configuration update across Apollo and TEMP-1 device profiles.

ESPHome 2026.8.0 replaces rgb_order/is_rgbw/is_wrgb with a single
channel_colors key on esp32_rmt_led_strip (esphome#18474).
@coderabbitai

coderabbitai Bot commented Aug 28, 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 Plus

Run ID: cbfa220a-14a1-4704-9c3b-560119f37eb7

📥 Commits

Reviewing files that changed from the base of the PR and between 23120b0 and 75355cb.

📒 Files selected for processing (8)
  • Integrations/ESPHome/Core.yaml
  • Integrations/ESPHome/TEMP-1.yaml
  • Integrations/ESPHome/TEMP-1B.yaml
  • Integrations/ESPHome/TEMP-1B_BLE.yaml
  • Integrations/ESPHome/TEMP-1B_Minimal.yaml
  • Integrations/ESPHome/TEMP-1_BLE.yaml
  • Integrations/ESPHome/TEMP-1_Beta.yaml
  • Integrations/ESPHome/TEMP-1_Minimal.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • Integrations/ESPHome/Core.yaml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The PR updates ESPHome minimum versions and firmware version substitutions. It also replaces the WS2812 LED rgb_order: grb setting with channel_colors: GRB in two configurations.

Changes

ESPHome configuration updates

Layer / File(s) Summary
Update ESPHome version requirements
Integrations/ESPHome/*.yaml
Configurations raise min_version from 2025.11.0 to 2026.8.0. Core.yaml and TEMP-1_Beta.yaml update their firmware version substitutions to 26.8.27.1.
Update RGB channel ordering
Integrations/ESPHome/Core.yaml, Integrations/ESPHome/TEMP-1_Beta.yaml
The WS2812 esp32_rmt_led_strip configurations replace rgb_order: grb with channel_colors: GRB.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 75355

This updates ESPHome configuration syntax and minimum-version settings across the affected device configurations without identified behavior or integration risk; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: bharvey88

Poem

A rabbit updates version signs
And tunes the LED lines
Green, red, blue now hop in place
Two configs share the setting’s grace
New ESPHome rules are clear
The glowing path is ready here

🚥 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 and concisely describes the primary change: replacing deprecated rgb_order with channel_colors for ESPHome LED configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (8 skipped: 8 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch channel-colors-migration

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.

Bunton33 and others added 2 commits August 27, 2026 23:36
`channel_colors` only exists from ESPHome 2026.8.0 (esphome#18474), but
these YAMLs still declared an older min_version. Devices adopted through
the ESPHome dashboard track this repo at HEAD with no pinned ref, so on
an older ESPHome the next recompile fails with "Unknown option
channel_colors" pointing into a file the user never wrote, instead of a
clear version error.

ESPHome merges remote packages and checks min_version in the preload
phase, before component schemas load, so the bump produces the right
message. No firmware change: min_version is validation-only and emits no
code, so the compiled binary is unaffected.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@bharvey88 bharvey88 added the bugfix Bug fix label Aug 30, 2026

@bharvey88 bharvey88 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified against upstream rather than the description: channel_colors landed in ESPHome 2026.8.0 (esphome#18474), and rgb_order still validates but is marked for removal in 2027.3.0. validate_channel_colors uppercases its input and channel_colors_struct derives the same r/g/b indices, so GRB here is byte-identical to the previous grb. No behaviour change and no change to the compiled binary.

Every esp32_rmt_led_strip block in this repo is covered. I pushed one commit on top bumping min_version to 2026.8.0: devices adopted through the ESPHome dashboard track this repo at HEAD with no pinned ref, so without it an adopter on an older ESPHome would hit Unknown option channel_colors inside a file they never wrote. ESPHome merges remote packages and checks min_version before component schemas load, so the bump surfaces the right error instead.

CI green on stable, beta and dev.

@bharvey88
bharvey88 merged commit 4983129 into beta Aug 30, 2026
37 checks passed
@bharvey88
bharvey88 deleted the channel-colors-migration branch August 30, 2026 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants