Skip to content
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

Bandersnatch: ring-context generic over domain size #2581

Merged

Conversation

davxy
Copy link
Member

@davxy davxy commented Dec 1, 2023

Serialized length is now statically computed depending on the domain size.

Opens the primitive to more generic usages not related to Sassafras expectations

Address one point of #2364

@davxy davxy self-assigned this Dec 1, 2023
@davxy davxy added the T0-node This PR/Issue is related to the topic “node”. label Dec 1, 2023
@davxy davxy requested review from bkchr and a team December 1, 2023 18:12
@davxy
Copy link
Member Author

davxy commented Dec 1, 2023

@bkchr this should address your concern reported here #1577 (comment)

@paritytech paritytech deleted a comment from paritytech-cicd-pr Dec 1, 2023
@davxy davxy mentioned this pull request Dec 1, 2023
3 tasks
@davxy davxy requested a review from a team December 1, 2023 18:44
@michalkucharczyk michalkucharczyk requested a review from a team December 1, 2023 18:48
@davxy davxy enabled auto-merge (squash) December 1, 2023 18:55
@davxy davxy merged commit ecfdb2b into paritytech:master Dec 1, 2023
112 of 115 checks passed
@davxy davxy deleted the davxy-brandersnatch-flexible-domain-len branch December 2, 2023 14:06
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
Serialized length is now statically computed depending on the domain
size.

Opens the primitive to more generic usages not related to Sassafras
expectations

Address one point of
paritytech#2364
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T0-node This PR/Issue is related to the topic “node”.
Projects
Status: done
Development

Successfully merging this pull request may close these issues.

5 participants