-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Substrate Companion #7118
Substrate Companion #7118
Conversation
@@ -463,6 +463,7 @@ impl pallet_vesting::Config for Runtime { | |||
impl pallet_sudo::Config for Runtime { | |||
type RuntimeEvent = RuntimeEvent; | |||
type RuntimeCall = RuntimeCall; | |||
type WeightInfo = (); |
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.
we need to run the benches, and use an actual weight for of runtimes.
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.
Yeah, can we call a bot to do that like is done on the Substrate repo?
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.
yes, but I think it wont work until substrate is not merged
You can copy the weights from substrate. During the release process they will be replaced by actual weights.
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.
Thanks, will do that.
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.
Done for Rococo/Westend. For the testing runtimes it does not matter.
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.
Weights will be updated before the next release.
The CI pipeline was cancelled due to failure one of the required jobs. |
Waiting for commit status. |
Merge cancelled due to error. Error: Response error (status 500 Internal Server Error):
|
bot merge |
Waiting for commit status. |
paritytech/substrate#13880
Cumulus companion: paritytech/cumulus#2514