-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Display raw UINT/INT values as hex #821
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #821 +/- ##
==========================================
- Coverage 72.73% 72.72% -0.02%
==========================================
Files 472 472
Lines 30774 30774
Branches 878 878
==========================================
- Hits 22384 22379 -5
- Misses 8295 8303 +8
+ Partials 95 92 -3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
this.rawValue = '0x' + values[0][0].toString(16).toUpperCase() | ||
} else { | ||
this.rawValue = values[0][0] | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted hex to be in addition to the decimal display, not replace it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something like 0xA (10)
or 10 (0xA)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That or a separate line. Either would be fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Of those: 10 (0xA)
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
closes #775