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

chore(deps): bump github.com/centrifuge/go-substrate-rpc-client/v4 from 4.0.9 to 4.0.10 #3009

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Dec 19, 2022

Bumps github.com/centrifuge/go-substrate-rpc-client/v4 from 4.0.9 to 4.0.10.

Release notes

Sourced from github.com/centrifuge/go-substrate-rpc-client/v4's releases.

v4.0.10

Changelog

  • 8b830c55 → fix of account-info structure (#309)

v4.0.9..v4.0.10

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [github.com/centrifuge/go-substrate-rpc-client/v4](https://github.com/centrifuge/go-substrate-rpc-client) from 4.0.9 to 4.0.10.
- [Release notes](https://github.com/centrifuge/go-substrate-rpc-client/releases)
- [Commits](centrifuge/go-substrate-rpc-client@v4.0.9...v4.0.10)

---
updated-dependencies:
- dependency-name: github.com/centrifuge/go-substrate-rpc-client/v4
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/centrifuge/go-substrate-rpc-client/v4-4.0.10 branch from 4b4107b to a039d16 Compare January 2, 2023 15:11
Bumps [github.com/centrifuge/go-substrate-rpc-client/v4](https://github.com/centrifuge/go-substrate-rpc-client) from 4.0.9 to 4.0.10.
- [Release notes](https://github.com/centrifuge/go-substrate-rpc-client/releases)
- [Commits](centrifuge/go-substrate-rpc-client@v4.0.9...v4.0.10)

---
updated-dependencies:
- dependency-name: github.com/centrifuge/go-substrate-rpc-client/v4
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@qdm12
Copy link
Contributor

qdm12 commented Jan 2, 2023

TestSystemModule_AccountNextIndex seems to fail due to some decoding error (?), I digged a bit into it but can't really pin why/fix it 😢

@qdm12 qdm12 force-pushed the dependabot/go_modules/github.com/centrifuge/go-substrate-rpc-client/v4-4.0.10 branch from a039d16 to 7a0b471 Compare January 2, 2023 16:06
@EclesioMeloJunior EclesioMeloJunior self-assigned this Jan 3, 2023
…e-rpc-client/v4-4.0.10' of github.com:ChainSafe/gossamer into dependabot/go_modules/github.com/centrifuge/go-substrate-rpc-client/v4-4.0.10
@EclesioMeloJunior
Copy link
Member

EclesioMeloJunior commented Jan 3, 2023

TestSystemModule_AccountNextIndex seems to fail due to some decoding error (?), I digged a bit into it but can't really pin why/fix it 😢

In version 4.0.10 they included a new field in the types.AccountInfo, that new field introduces 4 new bytes to decode (U32 type) so we should update the test to include these new 4 bytes to avoid the error:

[...] Cannot read the required number of bytes 16, only 12 available

fixed here: c1ba8c7

edit

The same problem was happening in the test TestAuthorSubmitExtrinsic, just changed to use westend-dev runtime in the test. #2895 (comment)

@qdm12 fyi

@codecov
Copy link

codecov bot commented Jan 3, 2023

Codecov Report

Merging #3009 (d4a6de1) into development (3729f32) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Additional details and impacted files
@@               Coverage Diff               @@
##           development    #3009      +/-   ##
===============================================
- Coverage        51.86%   51.86%   -0.01%     
===============================================
  Files              219      219              
  Lines            27718    27720       +2     
===============================================
  Hits             14376    14376              
- Misses           12115    12117       +2     
  Partials          1227     1227              

@EclesioMeloJunior EclesioMeloJunior force-pushed the dependabot/go_modules/github.com/centrifuge/go-substrate-rpc-client/v4-4.0.10 branch from 904cefc to d4a6de1 Compare January 4, 2023 17:46
@EclesioMeloJunior EclesioMeloJunior merged commit 58ce668 into development Jan 4, 2023
@EclesioMeloJunior EclesioMeloJunior deleted the dependabot/go_modules/github.com/centrifuge/go-substrate-rpc-client/v4-4.0.10 branch January 4, 2023 19:17
Copy link

🎉 This PR is included in version 0.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

None yet

3 participants