-
Notifications
You must be signed in to change notification settings - Fork 1.6k
cargo: sync workspace members with dependencies #4579
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs a merge with master and looks good other than a question about bridges.
Cargo.toml
Outdated
"bridges/bin/runtime-common", | ||
"bridges/modules/dispatch", | ||
"bridges/modules/grandpa", | ||
"bridges/modules/messages", | ||
"bridges/primitives/chain-rococo", | ||
"bridges/primitives/chain-wococo", | ||
"bridges/primitives/header-chain", | ||
"bridges/primitives/message-dispatch", | ||
"bridges/primitives/messages", | ||
"bridges/primitives/polkadot-core", | ||
"bridges/primitives/runtime", | ||
"bridges/primitives/test-utils", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about bridges pallets as it's added as a subtree and was removed here last time by @HCastano
cc @svyatonik @acatangiu
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I already discussed this with @FlorianFranzen, but he still doesn't has removed them..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imo there's no harm in adding these crates to workspace - they're built with the code anyways. We should avoid adding all bridges/*
crates - there are crates that shall not be built here (ideally we should remove them from the polkadot repo, but ...).
* cargo: sync workspace members with dependencies * Remove bridges stuff * Fix Co-authored-by: Bastian Köcher <info@kchr.de>
@bkchr Thank you, not sure why this dropped off my notifications. |
This syncs the workspace crates returned by
cargo metadata
with the crates insideCargo.toml
, which resolves some issues I was having with naersk as well with some older lsp servers.