-
Notifications
You must be signed in to change notification settings - Fork 87
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
Remove NodeToNodeV_12 older #4985
Remove NodeToNodeV_12 older #4985
Conversation
1da0444
to
0a75b40
Compare
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
ouroboros-network/sim-tests-lib/Ouroboros/Network/BlockFetch/Examples.hs
Outdated
Show resolved
Hide resolved
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 apart from the encoding/decoding of older versions.
ouroboros-network-api/src/Ouroboros/Network/NodeToNode/Version.hs
Outdated
Show resolved
Hide resolved
0a75b40
to
a20463a
Compare
a20463a
to
2dee544
Compare
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
2dee544
to
cd65b5a
Compare
cd65b5a
to
d2d90a8
Compare
d2d90a8
to
9e45f34
Compare
Description
Removes support for NodeToNodeV_12 and older as these are unable to cross the hard fork boundary.
Checklist
Quality
Maintenance
ouroboros-network
project.