Skip to content

Conversation

@Elwell
Copy link
Contributor

@Elwell Elwell commented Oct 11, 2025

Description:

This page describes the 'esphome' section so doesn't need to include the esp32 section which itself includes the old 'board' line rather than the preferred 'variant' setting

Checklist:

  • I am merging into next because this is new documentation that has a matching pull-request in esphome as linked above.
    or

  • I am merging into current because this is a fix, change and/or adjustment in the current documentation and is not for a new component or feature.

  • Link added in /components/index.rst when creating new documents for new components or cookbook.

This page describes the 'esphome' section so doesn't need to include
the esp32 section which itself includes the old 'board' line rather than
the preferred 'variant' setting
@esphome esphome bot added the current label Oct 11, 2025
@netlify
Copy link

netlify bot commented Oct 11, 2025

Deploy Preview for esphome ready!

Name Link
🔨 Latest commit da8547c
🔍 Latest deploy log https://app.netlify.com/projects/esphome/deploys/68f1a53b9ef1ff00085b5a43
😎 Deploy Preview https://deploy-preview-5472--esphome.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 11, 2025

Walkthrough

Removed the ESP32 board specification lines from an ESPHome YAML example in documentation, deleting the esphome section’s esp32/board configuration. No code or control-flow changes.

Changes

Cohort / File(s) Summary of changes
Docs: ESPHome example config
content/components/esphome.md
Deleted YAML lines defining esp32: and board: nodemcu-32s from a config example; no alternative configuration added.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

current

Suggested reviewers

  • DjordjeMandic
  • jesserockz

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title concisely captures the main change by stating the removal of an unrelated esp32 stanza from example code, which is the primary modification in the diff. It is specific and clear enough that a reviewer understands the core change at a glance.
Description Check ✅ Passed The pull request description clearly explains why the esp32 stanza is removed and provides context about the 'esphome' section and board variant, making it directly related to the changeset. It adequately describes the intent of the documentation adjustment.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between edcf186 and 1cd27d2.

📒 Files selected for processing (1)
  • content/components/esphome.md (0 hunks)
💤 Files with no reviewable changes (1)
  • content/components/esphome.md

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 and usage tips.

@swoboda1337 swoboda1337 merged commit 87f9442 into esphome:current Oct 17, 2025
3 of 4 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants