Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Co #5091: Runtime dependent weights for parachains #1081

Merged
merged 4 commits into from
Mar 12, 2022

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Mar 11, 2022

Instead of just using the polkadot-runtime weights in the parachains 🤦 actually copy them in there.

Companion for paritytech/polkadot#5091

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez added A3-inprogress B0-silent Changes should not be mentioned in any release notes labels Mar 11, 2022
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez changed the title Runtime dependent weights for parachains Co #5091: Runtime dependent weights for parachains Mar 11, 2022
// See the License for the specific language governing permissions and
// limitations under the License.

pub mod constants {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand all these extra constants modules. Why do we need them?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't, it is just a naming convention from Substrate.
Does not really makes tbh, so I will remove them from the weight templates.

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 99bb22d into master Mar 12, 2022
@paritytech-processbot paritytech-processbot bot deleted the oty-parachain-weights branch March 12, 2022 20:12
bkchr added a commit that referenced this pull request Mar 22, 2022
jiguantong added a commit to darwinia-network/darwinia-parachain that referenced this pull request Jun 1, 2022
jiguantong added a commit to darwinia-network/darwinia-parachain that referenced this pull request Jun 1, 2022
jiguantong added a commit to darwinia-network/darwinia-parachain that referenced this pull request Jun 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants