Add unit µA to eletric current#3025
Add unit µA to eletric current#3025Phunkafizer wants to merge 1 commit intohome-assistant:masterfrom
Conversation
📝 WalkthroughWalkthroughDocumentation updates to the CURRENT device class in both number and sensor entity docs, adding µA (microamps) to the list of supported units alongside existing A and mA units. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/core/entity/number.md`:
- Line 44: The docs entry for NumberDeviceClass.CURRENT incorrectly lists the
microamp unit (µA); edit docs/core/entity/number.md to remove "µA" from the
CURRENT device class supported units (leave only "A, mA"), and search for any
other occurrences of NumberDeviceClass.CURRENT in the same file to remove µA
mentions so docs match Home Assistant Core implementation and the closed issue
status.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 2dfd1dc7-41f0-4024-9bb4-e8b5900962cc
📒 Files selected for processing (2)
docs/core/entity/number.mddocs/core/entity/sensor.md
| | `NumberDeviceClass.CO` | ppb, ppm, µg/m³, mg/m³ | Concentration of carbon monoxide. | ||
| | `NumberDeviceClass.CONDUCTIVITY` | S/cm, mS/cm, µS/cm | Conductivity | ||
| | `NumberDeviceClass.CURRENT` | A, mA | Current | ||
| | `NumberDeviceClass.CURRENT` | A, mA, µA | Current |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
Does Home Assistant Core support microamps (µA) as a unit for the CURRENT device class in version implemented by PR 166786?
💡 Result:
No, Home Assistant Core does not support microamps (µA) as a unit for the CURRENT device class in the version implemented by PR 166786. No such PR exists; searches returned no results for PR 166786 in the home-assistant/core repository. Official documentation lists only A and mA as supported units for current. A feature request (issue #161400) to add µA was closed as "not planned" in January 2026.
Citations:
- 1: https://www.home-assistant.io/components/sensor
- 2: add unit "µA" for sensor of deviceclass "current" core#161400
- 3: Add mA to SensorDeviceClass.CURRENT units core#84492
- 4: https://www.airwaymedical.net
Remove µA from CURRENT device class—not supported by Home Assistant Core.
Home Assistant Core does not support microamps (µA) as a unit for the CURRENT device class. Official documentation lists only A and mA as supported units. A feature request (issue #161400) to add µA support was closed as "not planned" in January 2026.
The documentation change contradicts the actual implementation and will mislead users. Remove µA from the supported units list.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/core/entity/number.md` at line 44, The docs entry for
NumberDeviceClass.CURRENT incorrectly lists the microamp unit (µA); edit
docs/core/entity/number.md to remove "µA" from the CURRENT device class
supported units (leave only "A, mA"), and search for any other occurrences of
NumberDeviceClass.CURRENT in the same file to remove µA mentions so docs match
Home Assistant Core implementation and the closed issue status.
Proposed change
Type of change
Checklist
Additional information
Add unit 'µA' for the units of electric current core#166786
Summary by CodeRabbit