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

[polkadart_cli] generate method for getting a storage's key #376

Merged

Conversation

clangenb
Copy link
Contributor

I consider this as an intermediate step to generating code for storage subscriptions, which will take me some more time to learn about generating code before I could implement this. I am not sure how useful it is on its own yet. Regardless, I thought I will file a PR and let you judge. :)

Copy link

codecov bot commented Nov 23, 2023

Codecov Report

Merging #376 (9c015cb) into main (5916094) will decrease coverage by 0.11%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #376      +/-   ##
==========================================
- Coverage   44.84%   44.73%   -0.11%     
==========================================
  Files         146      146              
  Lines        8230     8250      +20     
==========================================
  Hits         3691     3691              
- Misses       4539     4559      +20     
Flag Coverage Δ
polkadart 17.31% <ø> (ø)
polkadart_cli 12.19% <0.00%> (-0.09%) ⬇️
polkadart_keyring 94.25% <ø> (ø)
polkadart_scale_codec 54.93% <ø> (ø)
ss58 96.87% <ø> (ø)
substrate_bip39 64.34% <ø> (ø)
substrate_metadata 87.61% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...ckages/polkadart_cli/lib/src/generator/pallet.dart 0.00% <0.00%> (ø)

@leonardocustodio
Copy link
Owner

If you have some free time would you mind taking a look at this one @Lohann ?

@clangenb clangenb force-pushed the cl/generate-subscribe-storage branch from 3380de9 to 9c015cb Compare November 29, 2023 01:51
@leonardocustodio leonardocustodio merged commit de88d36 into leonardocustodio:main Nov 29, 2023
8 checks passed
@leonardocustodio leonardocustodio self-assigned this Nov 29, 2023
@leonardocustodio
Copy link
Owner

Released under polkadart_cli 0.2.4 thank you very much for your contribution @clangenb

@clangenb clangenb deleted the cl/generate-subscribe-storage branch November 30, 2023 00:18
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