-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
refactor: reuse utils added in block production race refactor #6422
Conversation
*/ | ||
export function prettyWeiToEth(wei: bigint, suffix = false): string { |
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.
Removed option to omit suffix as it is unused and would just lead to inconsistent logs.
We can discuss how we wanna log ETH values (with / without suffix), I am fine with either as long as we do it consistently.
@@ -74,11 +74,6 @@ export function wrapPromise<T>(promise: PromiseLike<T>): PromiseResult<T> { | |||
return result; | |||
} | |||
|
|||
/** | |||
* ArrayToTuple converts an `Array<T>` to `[T, ...T]` |
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.
this comment seemed misplaced
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6422 +/- ##
============================================
- Coverage 61.72% 61.71% -0.01%
============================================
Files 553 553
Lines 57856 57839 -17
Branches 1829 1829
============================================
- Hits 35711 35696 -15
+ Misses 22108 22106 -2
Partials 37 37 |
Performance Report✔️ no performance regression detected 🚀🚀 Significant benchmark improvement detected
Full benchmark results
|
🎉 This PR is included in v1.16.0 🎉 |
Motivation
De-duplicate some code, consistent logging across bn and vc
Description
#6241 added a util to format block values. This PR just reuses it on the vc side to ensure we have consistent logging.