-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Align Types between Bulletin and SDK #10578
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: master
Are you sure you want to change the base?
Conversation
|
/cmd prdoc --audience runtime_dev --bump minor |
…time_dev --bump minor'
| // TODO: find out what is "allocator" and "allocator limit" | ||
| // Setting higher limit also requires raising the allocator limit. | ||
| // TODO: not used, can we remove or use? | ||
| pub const DEFAULT_MAX_TRANSACTION_SIZE: u32 = 8 * 1024 * 1024; | ||
| // TODO: not used, can we remove or use? | ||
| pub const DEFAULT_MAX_BLOCK_TRANSACTIONS: u32 = 512; |
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.
Can you create an issue for figuring this out and include the issue link on these comments? That way it won't fall through the cracks
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.
Created: #10591
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.
Updated TODO: fa44622
The PR aligns types between Bulletin and SDK.
Addresses paritytech/polkadot-bulletin-chain#86
Relates to
StoragePeriodbetween Bulletin and SDK polkadot-bulletin-chain#134RuntimeDebugand replace it withDebug#10582