-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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.
LGTM
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
@@ -18,7 +18,7 @@ | |||
//! Autogenerated weights for pallet_balances | |||
//! | |||
//! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 2.0.0 | |||
//! DATE: 2021-01-06, STEPS: [50, ], REPEAT: 20, LOW RANGE: [], HIGH RANGE: [] | |||
//! DATE: 2021-01-06, STEPS: \[50, \], REPEAT: 20, LOW RANGE: [], HIGH RANGE: [] |
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.
where is this change coming from?
do you need to update the weight template / output code?
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 we probably need to update the templates,
rust doc complains the content inside []
is not a rust type
bot merge |
Waiting for commit status. |
Checks failed; merge aborted. |
part of #7882
migrate pallet-example, pallet-example-offchain-worker and pallet-example-parallel to pallet macro.
the doc in pallet-example is slightly changed. Also it might be good to also have inherent, validate unsigned, and error docs but this is for later PR.
At some point I think the doc in frame_support::pallet and pallet-example should be merged