-
Notifications
You must be signed in to change notification settings - Fork 90
add NodeToClientV_23 -> GetDRepsDelegations #5222
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
base: main
Are you sure you want to change the base?
Conversation
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.
No need to update cardano-ping, we'll release it along side #5205.
|
You need to update |
|
Also a cddl test failed: You just need to add the new version to the spec. |
|
@coot would you be kind enough to do those remaining updates yourself 🙏? It'll likely take you far less time than me. |
|
Nevermind; |
|
sure, I'll take care of it. There are also some conflicts which are likely due to a refactoring PR we merged just recently. |
|
I cannot becasue it's a PR from a fork. You can update your branch using |
|
Update to what @coot? The changes for the spec & orphan instance are already on my branch and yours. |
|
You only updated a comment in the spec, not the spec itself. That's why the test still fails. |
|
🤦♂️ |
|
Done @coot |
Description
Add support for a newly introduce ledger query, that still needs to be wired in the state-query protocol.
See also IntersectMBO/ouroboros-consensus#1717.
Checklist
Quality
New tests are added and existing tests are updated.Maintenance
ouroboros-networkproject.