-
Notifications
You must be signed in to change notification settings - Fork 115
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
runtime: Support for block tags #3848
Conversation
c91c105
to
7a37900
Compare
Codecov Report
@@ Coverage Diff @@
## master #3848 +/- ##
==========================================
+ Coverage 66.99% 67.06% +0.07%
==========================================
Files 404 404
Lines 41467 41467
==========================================
+ Hits 27779 27811 +32
+ Misses 9740 9708 -32
Partials 3948 3948
Continue to review full report at Codecov.
|
c54f9e1
to
08d749b
Compare
08d749b
to
30c014d
Compare
@@ -0,0 +1 @@ | |||
runtime: Add support for block tags |
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.
Why is this breaking?
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.
Hm i was thinking to mark it as breaking as ExecuteTxResult
(publicly exposed runtime api struct) was changed. I guess it depends on what definition of "breaking" we'd like to use for these fragments :-)
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.
Oh, yeah that's true, keep it as breaking.
Runtime: Add support for block tags (tags not emitted by a specific transaction).
Needed for: oasis-sdk#consensus-module (oasisprotocol/oasis-sdk#92)