fix(facts): use RPM shortDesc for ESC rotation rate#14634
Conversation
Match common telemetry labeling; units remain implied by value.
Build ResultsPlatform Status
All builds passed. Pre-commit
Pre-commit hooks: 0 passed, 0 failed, 0 skipped. Test Resultslinux-coverage-integration: 31 passed, 0 skipped Code CoverageCoverage: 67.6% No baseline available for comparison Artifact Sizes
Updated: 2026-07-18 12:25:42 UTC • Commit: 339df65 • Triggered by: Android |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #14634 +/- ##
==========================================
+ Coverage 25.47% 31.49% +6.02%
==========================================
Files 769 785 +16
Lines 65912 66935 +1023
Branches 30495 31023 +528
==========================================
+ Hits 16788 21082 +4294
+ Misses 37285 31539 -5746
- Partials 11839 14314 +2475
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
|
Consolidating per @DonLakeFlyer's request into a single PR: #14651. Closing this one. Thanks! |
What
Replace EscStatus
rpmshortDesc "Rotation Per Minute" with the standard abbreviation RPM.Motivation
Telemetry panels are dense; RPM is universal and matches generator fact style (
units: rpm).Verification
JSON-only metadata; no unit conversion change.
Reviewed by hand on master tip.