Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Extract RPC definitions from RPC crate. #3502

Merged
merged 7 commits into from
Aug 29, 2019
Merged

Conversation

tomusdrw
Copy link
Contributor

Currently the trait definitions of RPC APIs (and the client implementations) are inside the substrate-rpc crate that also contains server-side implementations.

As the consequence the proejcts like ink or subxt (see paritytech/subxt#13) need to compile a large chunk of substrate to get the client part.

This PR extracts the APIs to a separate substrate-rpc-apis crate, which should maintain a minimal set of dependencies (like primitives).

@tomusdrw tomusdrw added A0-please_review Pull request needs code review. M6-rpcapi labels Aug 28, 2019
Copy link
Member

@shawntabrizi shawntabrizi left a comment

Choose a reason for hiding this comment

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

LGTM

@shawntabrizi shawntabrizi added A8-looksgood and removed A0-please_review Pull request needs code review. labels Aug 28, 2019
@tomusdrw tomusdrw mentioned this pull request Aug 29, 2019
3 tasks
@svyatonik svyatonik merged commit 95fc421 into master Aug 29, 2019
@svyatonik svyatonik deleted the td-rpc-definitions branch August 29, 2019 08:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants