Skip to content

test-runtime: json_patch dep removed#3062

Merged
bkchr merged 1 commit intomasterfrom
mku-json-patch-dep-removed
Jan 25, 2024
Merged

test-runtime: json_patch dep removed#3062
bkchr merged 1 commit intomasterfrom
mku-json-patch-dep-removed

Conversation

@michalkucharczyk
Copy link
Copy Markdown
Contributor

We don't need dependency on json_patch there is implementation of needed function in sc_chain_spec crate.
This PR makes this function publicly available and removes the dep.

GenesisBlockBuilder,
},
genesis_config_builder::GenesisConfigBuilderRuntimeCaller,
json_patch::merge as json_merge,
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.

Maybe not most elegant solution, but I don't want to add crate for this single tiny merge function. Let me know if it is ok.

@bkchr bkchr added the R0-no-crate-publish-required The change does not require any crates to be re-published. label Jan 25, 2024
@bkchr bkchr enabled auto-merge January 25, 2024 12:55
@michalkucharczyk michalkucharczyk requested a review from a team January 25, 2024 13:05
@bkchr bkchr added this pull request to the merge queue Jan 25, 2024
Merged via the queue into master with commit 73fd8cd Jan 25, 2024
@bkchr bkchr deleted the mku-json-patch-dep-removed branch January 25, 2024 15:52
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
We don't need dependency on `json_patch` there is implementation of
needed function in `sc_chain_spec` crate.
This PR makes this function publicly available and removes the dep.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R0-no-crate-publish-required The change does not require any crates to be re-published.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants