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

Introduce 'BlockImportParams::insert_intermediate' method #12215

Merged
merged 1 commit into from
Sep 21, 2022
Merged

Introduce 'BlockImportParams::insert_intermediate' method #12215

merged 1 commit into from
Sep 21, 2022

Conversation

davxy
Copy link
Member

@davxy davxy commented Sep 8, 2022

Nothing special here... Only a helper method to allow to ignore the way intermediates are stored within the BlockImportParams.

The user is still free to use directly the container as before if for some reason it already has a Cow (even though nobody in our codebase requires to use it in this way).

Renaming from 'intermediate_take' to 'intermediate_remove'
@davxy davxy requested a review from a team September 8, 2022 12:34
@davxy davxy self-assigned this Sep 8, 2022
@davxy davxy added 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. labels Sep 8, 2022
@davxy davxy requested review from koute and skunert September 9, 2022 06:18
@andresilva
Copy link
Contributor

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 9df7ac8 into paritytech:master Sep 21, 2022
ordian added a commit that referenced this pull request Sep 23, 2022
* master:
  [Fix] parameter_types! dead code errors (#12340)
  [Feature] Sequential migration execution for try-runtime (#12319)
  bench: Use `_` instead  of `::` in auto-generated file names (#12332)
  Fast Unstake Pallet (#12129)
  Rename anonymous to pure proxy (#12283)
  Migrate remaining old decl_* macros to the new pallet attribute macros (#12271)
  pallet-utility: Disallow none origin (#12321)
  Make automatic storage deposits resistant against changing deposit prices (#12083)
  Format templates and fix `--steps` default value (#12286)
  Bump `wasmtime` to 1.0.0 (#12317)
  Introduce 'intermediate_insert' method to hide implementation details (#12215)
  Bound staking storage items (#12230)
  Use `array-bytes` for All Array/Bytes/Hex Operations (#12190)
  BREAKING: Rename Origin (#12258)
  Use temporary db for benchmarking (#12254)
  rpc: Implement `chainSpec` RPC API (#12261)
  Import target block body during warp sync (#12300)
  Proper naming wrt expectations (#12311)
  [ci] Revert cancel-pipeline job (#12309)
@davxy davxy deleted the davxy/intermediate-refactor branch September 24, 2022 15:00
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants