-
Notifications
You must be signed in to change notification settings - Fork 224
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
rpc: Replace RPC ABCI types with ABCI domain types #1092
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1092 +/- ##
========================================
+ Coverage 64.7% 64.8% +0.1%
========================================
Files 240 238 -2
Lines 20789 20730 -59
========================================
- Hits 13464 13453 -11
+ Misses 7325 7277 -48
Continue to review full report at Codecov.
|
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
…ermint::Hash Signed-off-by: Thane Thomson <connect@thanethomson.com>
2145c2b
to
dddae46
Compare
Hey @thanethomson , this would be helpful to get this merged for implementing penumbra-zone/penumbra#1232. I have the bandwidth to fork this branch and complete the PR, if you want btw! |
I don't think this can be merged, because the entire ABCI domain types work has to be rebuilt on the new |
Superseded by #1204 |
Closes #1090 and #947
.changelog/