Except for some very minor style issues, this looks like a good change! Thanks for the submission.
|1328 ↗||(On Diff #65799)|
Follow the style of the rest of the file please.
|27 ↗||(On Diff #65799)|
This should be in alphabetical order like the other variables (style).
This looks generally good, one minor comment and I think it's ready to go.
|384 ↗||(On Diff #65853)|
It would be nice to have a quick comment here saying that only batteries reported via ACPI are counted. Anybody still running APM won't have these numbers, but that's totally fine: APM is likely to be deleted soon anyway as laptops that require you use APM aren't really viable anymore as laptops... I'm also unsure how the 'pinebook' reports this information, but that too is a niche thing that's kinda weird still.... So just a quick comment will suffice.