-
Notifications
You must be signed in to change notification settings - Fork 983
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
state/write_log: refactor temp key-vals for better type-safety #3110
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3110 +/- ##
=======================================
Coverage 59.39% 59.40%
=======================================
Files 298 298
Lines 92771 92717 -54
=======================================
- Hits 55104 55077 -27
+ Misses 37667 37640 -27 ☔ View full report in Codecov by Sentry. |
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.
Minor comment but looks good to me, thank you!
let me know if this is rdy for merge or will have another change |
791d172
to
8ca77c8
Compare
should be good to go now |
172ba75
to
db5f425
Compare
* tomas/write-log-t-safety: changelog: add #3110 state/write_log: refactor temp key-vals for better type-safety
* tomas/write-log-t-safety: changelog: add #3110 state/write_log: refactor temp key-vals for better type-safety
Describe your changes
closes #2915
Indicate on which release or other PRs this topic is based on
v0.33.0
Checklist before merging to
draft