-
Notifications
You must be signed in to change notification settings - Fork 51
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
currency.Coin extension #1351
currency.Coin extension #1351
Conversation
Codecov Report
@@ Coverage Diff @@
## staging #1351 +/- ##
===========================================
- Coverage 29.82% 29.81% -0.01%
===========================================
Files 357 357
Lines 58619 58720 +101
===========================================
+ Hits 17481 17506 +25
- Misses 39186 39236 +50
- Partials 1952 1978 +26
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Manual system tests [success] with the following config
|
- block.GetScore updated - TransactionFeeStats renamed to FeeStats - chain.updateFeeStats method updated - wallet methods updated - transaction.Fee updated to coin
- configimpl fixed
Manual system tests [failure] with the following config
|
Manual system tests [failure] with the following config
|
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.
lgtm
Fixes
Changes
Need to be mentioned in CHANGELOG.md?
Tests
Tasks to complete before merging PR:
Associated PRs (Link as appropriate):