community-usermods: add Sensors I2C#337
Conversation
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>
WalkthroughA single row is added to the Community Usermods index table in Community Usermods Docs Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches✨ Simplify 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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/advanced/community-usermods.md (1)
33-33: 📐 Maintainability & Code Quality | 🔵 TrivialLGTM! 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/HAtoMQTT/Home Assistantfor 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
📒 Files selected for processing (1)
docs/advanced/community-usermods.md
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 /
/jsonand optional MQTT/Home Assistant, and can drive WLED's overall brightness from ambient light.wled-usermodtopic, MIT licensed, releasev1.0.0.wled00/changes); consumed via the git-URLcustom_usermodsmechanism. Verified building into WLED for ESP32.Single-line addition to the community usermods table, following the documented format.
Summary by CodeRabbit