Add SensorDeviceClass.UPTIME documentation#3032
Add SensorDeviceClass.UPTIME documentation#3032chemelli74 wants to merge 2 commits intohome-assistant:masterfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughAdded Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 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.
Pull request overview
Adds documentation for the newly introduced SensorDeviceClass.UPTIME in the sensor entity device class table, aligning developer docs with the corresponding Home Assistant Core change.
Changes:
- Document
SensorDeviceClass.UPTIMEin the “Available device classes” table. - Describe the intended meaning of the new device class (uptime / last boot time).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/core/entity/sensor.md (1)
85-85: Polish wording for grammar and consistency.Nice addition. The sentence should be grammatically corrected (
Represent→Represents) and can be phrased to match nearby device class descriptions.Proposed wording tweak
-| `SensorDeviceClass.UPTIME` | | Timestamp. Represent datetime of last boot of the device. +| `SensorDeviceClass.UPTIME` | | Timestamp. Represents the datetime of the device's last boot.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/core/entity/sensor.md` at line 85, Fix the grammar and match surrounding device class wording for SensorDeviceClass.UPTIME by changing "Represent datetime of last boot of the device." to a clear, consistent phrase such as "Represents the datetime of the device's last boot." or "Represents the datetime when the device last booted." Update the sentence to use "Represents" and the possessive form to align with nearby device class descriptions.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@docs/core/entity/sensor.md`:
- Line 85: Fix the grammar and match surrounding device class wording for
SensorDeviceClass.UPTIME by changing "Represent datetime of last boot of the
device." to a clear, consistent phrase such as "Represents the datetime of the
device's last boot." or "Represents the datetime when the device last booted."
Update the sentence to use "Represents" and the possessive form to align with
nearby device class descriptions.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 54fbfe1d-11f0-45d2-a349-0f74964138f8
📒 Files selected for processing (1)
docs/core/entity/sensor.md
abmantis
left a comment
There was a problem hiding this comment.
Can be merged when the core PR is merged.
Proposed change
Add details for the new device class
SensorDeviceClass.UPTIMEType of change
Checklist
Additional information
Summary by CodeRabbit