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

Fix storage info for decl_storage #9274

Merged
1 commit merged into from
Jul 6, 2021
Merged

Fix storage info for decl_storage #9274

1 commit merged into from
Jul 6, 2021

Conversation

gui1117
Copy link
Contributor

@gui1117 gui1117 commented Jul 5, 2021

decl_storage still doesn't generate storage info (in case of partial storage info).

I missed to remove the early return in the macro in #9246

This fix it and test it is correct.

@gui1117 gui1117 requested review from shawntabrizi and KiChjang July 5, 2021 11:57
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Jul 5, 2021
@gui1117 gui1117 added 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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Jul 5, 2021
@KiChjang
Copy link
Contributor

KiChjang commented Jul 6, 2021

bot merge

@ghost
Copy link

ghost commented Jul 6, 2021

Trying merge.

@ghost ghost merged commit 0f5a858 into master Jul 6, 2021
@ghost ghost deleted the gui-storage-info-fix branch July 6, 2021 01:58
dvdplm added a commit that referenced this pull request Jul 6, 2021
* master:
  fix staking version in genesis (#9280)
  fix storage info for decl_storage (#9274)
  Authority_discovery: expose assimilate_storage with GenesisBuild (#9279)
  Update CODEOWNERS (#9278)
  Remove in-tree `max-encoded-len` and use the new SCALE codec crate instead (#9163)
  bump a bunch of deps in parity-common (#9263)
  Bump linregress due to security vulnerability (#9262)
  pallet macro: always generate storage info on pallet struct (#9246)
  Less duplication in test code (#9270)
  Add `Chilled` event to staking chill extrinsics (#9250)
This pull request was closed.
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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants