Skip to content

chore: update deps to publish on crates.io#715

Merged
niklasad1 merged 9 commits intomainfrom
update-subxt-v0.32
Nov 23, 2023
Merged

chore: update deps to publish on crates.io#715
niklasad1 merged 9 commits intomainfrom
update-subxt-v0.32

Conversation

@niklasad1
Copy link
Copy Markdown
Contributor

@niklasad1 niklasad1 commented Nov 22, 2023

Preparation for a new release which mainly updates subxt and jsonrpsee.

One nice thing is that default port number in the URI can be omitted now

@niklasad1 niklasad1 changed the title WIP: update deps to crates.io chore: update deps to publish on crates.io Nov 23, 2023
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I was forced to update the metadata for the codegen to work

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hmm interesting, might be worth a subxt issue probably we'd want to keep backwards compatibility between 14 and 15 versions 🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I decided not to trim the metadata because of paritytech/subxt#1286

use subxt::backend::rpc::RpcClient as RawRpcClient;

/// Wraps the subxt interfaces to make it easy to use for the staking-miner.
#[derive(Clone, Debug)]
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

subxt's APIs has been modified a bit, thus this type is introduced to provide a wrapper for the APIs that the staking miner needs.

@niklasad1 niklasad1 merged commit 0f274c7 into main Nov 23, 2023
@niklasad1 niklasad1 deleted the update-subxt-v0.32 branch November 23, 2023 12:40
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