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

[Fix] Benchmark issue in staking #12290

Merged
merged 1 commit into from
Sep 18, 2022
Merged

[Fix] Benchmark issue in staking #12290

merged 1 commit into from
Sep 18, 2022

Conversation

ruseinov
Copy link
Contributor

@ruseinov ruseinov commented Sep 18, 2022

Blocks paritytech/polkadot#5930 and potentially all the polkadot builds that depend on latest substrate master.

polkadot companion: paritytech/polkadot#5930
^ not really, but we need it for the dependent build to pass

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Sep 18, 2022
@ruseinov ruseinov added E9-blocker ⛔️ C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. B0-silent Changes should not be mentioned in any release notes labels Sep 18, 2022
@bkchr
Copy link
Member

bkchr commented Sep 18, 2022

And you are sure that this isn't used in the benchmarks? Aka will not panic?

@ruseinov
Copy link
Contributor Author

And you are sure that this isn't used in the benchmarks? Aka will not panic?

It's not, mainly because those implementations are used for testing.

@ruseinov ruseinov merged commit 91b97b0 into master Sep 18, 2022
@ruseinov ruseinov deleted the ru/fix/bench-issue branch September 18, 2022 12:50
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
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. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants