Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make BlockNumberProvider::set_block_number available in tests / std #3974

Merged

Conversation

shawntabrizi
Copy link
Member

@shawntabrizi shawntabrizi commented Apr 3, 2024

This function is currently only exposed to runtime-benchmarks, where it should be available in all tests.

Also made it available in std because this is how set_block_number works in FRAME System, however, not sure if it is needed in the latest std/no_std paradigm.

substrate/primitives/runtime/src/traits.rs Outdated Show resolved Hide resolved
@bkchr bkchr added the R0-silent Changes should not be mentioned in any release notes label Apr 3, 2024
Co-authored-by: Bastian Köcher <git@kchr.de>
@ggwpez ggwpez enabled auto-merge April 4, 2024 13:47
@ggwpez ggwpez added this pull request to the merge queue Apr 4, 2024
Merged via the queue into paritytech:master with commit 9d052b7 Apr 4, 2024
130 of 133 checks passed
@shawntabrizi shawntabrizi deleted the shawntabrizi-set-block-number branch April 4, 2024 17:27
Ank4n pushed a commit that referenced this pull request Apr 9, 2024
…#3974)

This function is currently only exposed to runtime-benchmarks, where it
should be available in all tests.

Also made it available in `std` because this is how `set_block_number`
works in FRAME System, however, not sure if it is needed in the latest
std/no_std paradigm.

---------

Co-authored-by: Bastian Köcher <git@kchr.de>
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
dharjeezy pushed a commit to dharjeezy/polkadot-sdk that referenced this pull request Apr 9, 2024
…paritytech#3974)

This function is currently only exposed to runtime-benchmarks, where it
should be available in all tests.

Also made it available in `std` because this is how `set_block_number`
works in FRAME System, however, not sure if it is needed in the latest
std/no_std paradigm.

---------

Co-authored-by: Bastian Köcher <git@kchr.de>
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants