Skip to content

community-usermods: add Sensors I2C#337

Closed
AustinSaintAubin wants to merge 1 commit into
wled:mainfrom
AustinSaintAubin:add-sensors-i2c-usermod
Closed

community-usermods: add Sensors I2C#337
AustinSaintAubin wants to merge 1 commit into
wled:mainfrom
AustinSaintAubin:add-sensors-i2c-usermod

Conversation

@AustinSaintAubin

@AustinSaintAubin AustinSaintAubin commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Adds an index row for a community out-of-tree usermod: Sensors I2C.

It supports the common 3-in-1 I²C environmental breakout — HTU21D (temp/humidity), BMP180 (temp/pressure), BH1750FVI (lux) — reports to the Info page / /json and optional MQTT/Home Assistant, and can drive WLED's overall brightness from ambient light.

  • Repo is public, tagged with the wled-usermod topic, MIT licensed, release v1.0.0.
  • Self-contained (no wled00/ changes); consumed via the git-URL custom_usermods mechanism. Verified building into WLED for ESP32.

Single-line addition to the community usermods table, following the documented format.

Summary by CodeRabbit

  • Documentation
    • Added a new Community Usermod entry for Sensors I2C on esp32.
    • The listing includes supported I²C environmental sensors, licensing details, and notes about auto-brightness plus optional MQTT/Home Assistant integration.

Index the out-of-tree wled-usermod-sensors-i2c (HTU21D + BMP180 + BH1750FVI
3-in-1 I2C environmental sensor breakout; ambient-light auto-brightness, optional
MQTT/Home Assistant).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

A single row is added to the Community Usermods index table in docs/advanced/community-usermods.md, registering the "Sensors I2C" usermod for esp32 under the MIT license.

Community Usermods Docs Update

Layer / File(s) Summary
Sensors I2C table entry
docs/advanced/community-usermods.md
New row added to the usermods index table for "Sensors I2C" (HTU21D, BMP180, BH1750 sensors), with author link, esp32 platform, MIT license, and notes on WLED auto-brightness and MQTT/HA integration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • wled/WLED-Docs#312: Also modifies docs/advanced/community-usermods.md by adding/updating rows in the same Community Usermods index table.

Poem

🐇 A new sensor row hops into the table today,
I²C friends — HTU21D, BH1750 — hurray!
BMP180 joins the WLED brigade,
Brightness auto-tuned, MQTT relayed.
The docs grow longer, one small line at a time! 🌡️

🚥 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 is concise and accurately reflects the main change: adding the Sensors I2C community usermod entry.
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
✨ Simplify code
  • Create PR with simplified code

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.

@coderabbitai coderabbitai Bot 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.

🧹 Nitpick comments (1)
docs/advanced/community-usermods.md (1)

33-33: 📐 Maintainability & Code Quality | 🔵 Trivial

LGTM! The new table row follows the existing format and accurately reflects the usermod's capabilities as described in the PR objectives.

Optional: Consider expanding MQTT/HA to MQTT/Home Assistant for readers less familiar with the abbreviation, aligning with the guideline to avoid unexplained jargon.

🤖 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 `@docs/advanced/community-usermods.md` at line 33, The new community-usermods
table row is fine, but the “MQTT/HA” abbreviation may be unclear to some
readers. Update the Sensors I2C row in the community-usermods table to spell
this out as MQTT/Home Assistant while keeping the rest of the row format
unchanged.

Source: Coding guidelines

🤖 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 `@docs/advanced/community-usermods.md`:
- Line 33: The new community-usermods table row is fine, but the “MQTT/HA”
abbreviation may be unclear to some readers. Update the Sensors I2C row in the
community-usermods table to spell this out as MQTT/Home Assistant while keeping
the rest of the row format unchanged.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 15bbe8bd-d177-4815-9655-185b7b51c067

📥 Commits

Reviewing files that changed from the base of the PR and between f1b2e74 and 12a7745.

📒 Files selected for processing (1)
  • docs/advanced/community-usermods.md

@AustinSaintAubin AustinSaintAubin closed this by deleting the head repository Jul 2, 2026
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.

1 participant