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

types: Update the dispatch info weight #323

Merged
merged 1 commit into from
Jan 10, 2023
Merged

Conversation

cdamian
Copy link
Contributor

@cdamian cdamian commented Jan 10, 2023

No description provided.

@cdamian cdamian requested a review from mikiquantum January 10, 2023 13:27
@cdamian cdamian force-pushed the update-weight-to-struct branch from 33dfe27 to 179fe48 Compare January 10, 2023 14:00
@codecov
Copy link

codecov bot commented Jan 10, 2023

Codecov Report

Merging #323 (1e4e5df) into master (8b830c5) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #323      +/-   ##
==========================================
- Coverage   68.87%   68.77%   -0.10%     
==========================================
  Files         132      132              
  Lines        6791     6770      -21     
==========================================
- Hits         4677     4656      -21     
  Misses       1406     1406              
  Partials      708      708              
Impacted Files Coverage Δ
types/dispatch_result_with_post_info.go 66.66% <ø> (ø)
types/weight.go 100.00% <100.00%> (ø)
types/events.go 86.02% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@NunoAlexandre NunoAlexandre left a comment

Choose a reason for hiding this comment

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

Looks good, great that we can drop our custom Option 🚀

types/weight.go Outdated Show resolved Hide resolved
@cdamian cdamian force-pushed the update-weight-to-struct branch from 179fe48 to 1e4e5df Compare January 10, 2023 15:25
Copy link
Contributor

@mikiquantum mikiquantum left a comment

Choose a reason for hiding this comment

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

Thank you!

@cdamian cdamian merged commit 6b02d95 into master Jan 10, 2023
@cdamian cdamian deleted the update-weight-to-struct branch January 10, 2023 17:33
@cdamian cdamian mentioned this pull request Jan 10, 2023
MRamanenkau added a commit to Cerebellum-Network/go-substrate-rpc-client that referenced this pull request Feb 7, 2024
* client: Expose close function (centrifuge#311)

* fix of account-info structure (centrifuge#309)

Co-authored-by: Andriy <[email protected]>

* types: Update the dispatch info weight (centrifuge#323)

* types: Update anonymous proxy created event (centrifuge#324)

* change retarded uint8 to uint16 for networkid (centrifuge#339)

* fix: typos in `README` (centrifuge#348)

* fix: typos in `README`

* doc: add missing section

* Update event types (centrifuge#349)

* event: Update treasury events

* types: Add Orml Asset Registry events

* types: Update CurrencyID

* events: Adjust CustomMetadata for Orml events (centrifuge#354)

* Add events and extrinsics parsing logic based on metadata (centrifuge#326)

* events: Add initial metadata parsing logic

* events: Add override decoding funcs, add support for primitive arrays

* events: Extract decode type def

* events: Add event parsing test script, provide more details in error messages

* events: Add variant support for vectors

* milestone 1: Add type registry and test (centrifuge#327)

* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* registry: Add methods to create call and error registries

* dockerfile: Fix test Dockerfile

* registry: Add more tests

* Events parsing v2 milestone 2 (centrifuge#338)

* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* events: Add decoders for each event field type

* registry: Add decoders

* registry: Add retryable executor and use it in parser

* registry: Add mocks and more tests

* registry: Use BitVec when parsing bit sequences

* test: Add more mocks and tests

* registry: Update field name retrieval

* registry-test: Get headers instead of blocks

* Add generic chain RPC, more tests, and Dockerfiles for the 2nd milestone

* make: Add container name

* chain: Add constructor for default chain

* registry: Change field separator to

* parser: Add DefaultExtrinsicParser and DefaultExtrinsic

* retriever: Add DefaultExtrinsicRetriever and adjust tests

* rpc: Add more comments to default entities

* retriever: Enable all live tests

* Add registry readme (centrifuge#351)

* Fix docker run for w3f milestone 2 tests (centrifuge#353)

* Address review milestone 2 (centrifuge#355)

* add configurable event threshold

* Add natural language desc on event parsing

* make: Remove grant related targets

---------

Co-authored-by: Miguel Hervas <[email protected]>
Co-authored-by: Nikhil Ranjan <[email protected]>

* events: Add ORML tokens events (centrifuge#358)

* registry: Add overrides for field decoders (centrifuge#359)

* Change fields format to slice (centrifuge#360)

* registry: Add decoded fields as slice, add helper methods for retrieving values

* types: Remove unnecessary tests

* registry: Update return type of composite decoder to DecodedFields

* registry: Add ErrorID for ErrorRegistry, remove encode/decode for DecodedField

* registry: Add more tests for overrides

* registry: Reset storages when creating registries

* docs: Update godoc link (centrifuge#364)

* Polkadot update v0.9.43 (centrifuge#366)

* types: Update AccountInfo

* tests: Update tests

* client: Add CallContext to Client interface

* deps: Update go version, polkadot docker image

* lint: Obey

* deps: Update subkey to v2

* address: Return account ID creation error

* Remove extraneous output (centrifuge#343)

* Fix compilation errors

* Update to v8

---------

Co-authored-by: Cosmin Damian <[email protected]>
Co-authored-by: 1llusion1st <[email protected]>
Co-authored-by: Andriy <[email protected]>
Co-authored-by: Pierre <[email protected]>
Co-authored-by: Léo Vincent <[email protected]>
Co-authored-by: Miguel Hervas <[email protected]>
Co-authored-by: Nikhil Ranjan <[email protected]>
Co-authored-by: Felipe <[email protected]>
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.

3 participants