Skip to content

Pallet-anchors: replace Vec storage by BoundedVec#980

Merged
lemunozm merged 1 commit into
parachainfrom
anchors-bounded-vec
Sep 19, 2022
Merged

Pallet-anchors: replace Vec storage by BoundedVec#980
lemunozm merged 1 commit into
parachainfrom
anchors-bounded-vec

Conversation

@lemunozm

@lemunozm lemunozm commented Sep 19, 2022

Copy link
Copy Markdown
Contributor

Description

Replace Vec storage with BoundedVec in pallet-anchors to be prepared for Weights V2

Related to: #897

Changes and Descriptions

Modify Vec types found in the pallet-anchors storages.

Type of change

  • Bug fix (non-breaking change which fixes an issue). In this case, a future issue.

How Has This Been Tested?

cargo test -p pallet-anchors
cargo test --workspace --release --features test-benchmarks,try-runtime
cargo run --features runtime-benchmarks benchmark pallet --pallet="pallet-anchors" --chain="altair-dev" --extrinsic="*"

Checklist:

  • I have performed a self-review of my code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes
  • I rebased on the latest parachain branch

@lemunozm lemunozm changed the title Vec to BoundedVec Pallet-anchors: replace Vec storage by BoundedVec Sep 19, 2022
This was referenced Sep 19, 2022

@mustermeiszer mustermeiszer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@lemunozm lemunozm enabled auto-merge (squash) September 19, 2022 15:39
@lemunozm lemunozm merged commit d373aee into parachain Sep 19, 2022
@lemunozm lemunozm deleted the anchors-bounded-vec branch September 20, 2022 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants