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

Introduce RpcParams in sc-cli #5601

Merged
merged 6 commits into from
Sep 10, 2024
Merged

Conversation

liuchengxu
Copy link
Contributor

This PR is a pure refactoring by consolidating all the RPC related parameters into a dedicated RpcParams struct, allowing us at subcoin to build a custom run cmd without the need to duplicate code.

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable-int
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7266427

@liuchengxu liuchengxu marked this pull request as draft September 5, 2024 09:39
@niklasad1 niklasad1 added the T0-node This PR/Issue is related to the topic “node”. label Sep 5, 2024
@liuchengxu liuchengxu marked this pull request as ready for review September 5, 2024 11:35
@liuchengxu
Copy link
Contributor Author

@bkchr Could you also have a look at this small PR?

substrate/client/cli/src/params/rpc_params.rs Outdated Show resolved Hide resolved
substrate/client/cli/src/params/rpc_params.rs Outdated Show resolved Hide resolved
@niklasad1 niklasad1 enabled auto-merge September 10, 2024 14:17
@niklasad1 niklasad1 added this pull request to the merge queue Sep 10, 2024
Merged via the queue into paritytech:master with commit 278d1bf Sep 10, 2024
202 of 204 checks passed
mordamax pushed a commit to paritytech-stg/polkadot-sdk that referenced this pull request Sep 11, 2024
This PR is a pure refactoring by consolidating all the RPC related
parameters into a dedicated `RpcParams` struct, allowing us at subcoin
to build a custom run cmd without the need to duplicate code.

---------

Co-authored-by: Bastian Köcher <[email protected]>
Co-authored-by: Niklas Adolfsson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T0-node This PR/Issue is related to the topic “node”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants