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

Co #11456: Match statement fallthrough for benchmark commands #1350

Merged
merged 5 commits into from
Jul 19, 2022

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Jun 12, 2022

Currently each new benchmarking command in Substrate triggers a change in Cumulus.
After this MR there is no need for a hard-companion anymore, since the default behaviour will return an error for "unsupported command".

Useful for paritytech/substrate#11456 since adding the benchmark extrinsic logic itself is highly runtime-dependent and (currently) not useful to have.

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez added A0-pleasereview B0-silent Changes should not be mentioned in any release notes labels Jun 12, 2022
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Copy link
Contributor

@gilescope gilescope left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Copy link
Member Author

ggwpez commented Jul 19, 2022

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit f1b02db into master Jul 19, 2022
@paritytech-processbot paritytech-processbot bot deleted the oty-benchmarking-fallthrough branch July 19, 2022 12:54
jiguantong added a commit to darwinia-network/darwinia-parachain that referenced this pull request Sep 21, 2022
AurevoirXavier added a commit to darwinia-network/darwinia-parachain that referenced this pull request Oct 12, 2022
* Anchor polkadot-v0.9.27

* Companion for paritytech/cumulus#1436

* Companion for paritytech/cumulus#1421

* Companion for paritytech/cumulus#1350

* Companion for paritytech/cumulus#1456

* Use messages-substrate prepare-polkadot-v0.9.27 for debug

* Companion for darwinia-network/darwinia-messages-substrate#169

* Companion for darwinia-network/darwinia-messages-substrate#192

* Companion for darwinia-network/darwinia-messages-substrate#186

* Companion for darwinia-network/darwinia-messages-substrate#201

* Missing part companion for darwinia-network/darwinia-messages-substrate#169

* format

* replace prepare-polkadot-v0.9.27 -> polkadot-v0.9.27

* use `PANGOLIN_PANGOLIN_PARACHAIN_LANE` from `bridge_runtime_common` (#151)

* format

* Bump dependencies

Co-authored-by: Xavier Lau <xavier@inv.cafe>
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.

3 participants