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

Fix test full_block_council_election_cost #5787

Merged
merged 1 commit into from
Jul 18, 2022

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Jul 18, 2022

Reduce the constants of full_block_council_election_cost to unblock #5767
The new constants should have enough leeway and not error when we just update the weights.

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez requested a review from kianenigma July 18, 2022 09:50
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Jul 18, 2022
@paritytech-ci paritytech-ci requested a review from a team July 18, 2022 10:01
@ggwpez ggwpez merged commit 90d97b0 into coderobe/weights-26 Jul 18, 2022
@ggwpez ggwpez deleted the oty-council-election-cost-test branch July 18, 2022 10:38
paritytech-processbot bot pushed a commit that referenced this pull request Jul 28, 2022
* polkadot: update weights

* kusama: update weights

* westend: update weights

* rococo: update weights

* Reduce testing constants (#5787)

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
coderobe pushed a commit that referenced this pull request Aug 1, 2022
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
chevdor pushed a commit that referenced this pull request Aug 1, 2022
* westend: update weights

* kusama: update weights

* polkadot: update weights

* rococo: update weights

* kusama: readd phragmen remove_member_wrong_refund weight

* polkadot: readd phragmen remove_member_wrong_refund weight

* Reduce testing constants (#5787)

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants