Skip to content

Conversation

@DariuszDepta
Copy link
Member

@DariuszDepta DariuszDepta commented Aug 22, 2025

Dependencies between crates in packages directory are moved up to the workspace Cargo.toml.
This makes the maintenenace of dependencies easier, as they are placed in one file, like this:

[workspace.dependencies]
cosmwasm-core = { path = "./packages/core" }
cosmwasm-crypto = { path = "./packages/crypto" }
cosmwasm-derive = { path = "./packages/derive" }
cosmwasm-schema = { path = "./packages/schema" }
cosmwasm-schema-derive = { path = "./packages/schema-derive" }
cosmwasm-std = { path = "./packages/std", default-features = false }
cosmwasm-vm = { path = "./packages/vm" }
cosmwasm-vm-derive = { path = "./packages/vm-derive" }
cw-schema = { path = "./packages/cw-schema" }
cw-schema-derive = { path = "./packages/cw-schema-derive" }

P.S. To check which features are efectively used during dependency compilation use the following command:

$ cargo tree -e features

This PR was checked for dependency identity in the follwoing way:

  • on the main branch:
    $ git checkout main
    $ cargo tree -e features > a.txt
  • on the unified-dependencies branch:
    $ git checkout unified-dependencies
    $ cargo tree -e features > b.txt
$ diff -s a.txt b.txt

Output:

Files a.txt and b.txt are identical

@DariuszDepta DariuszDepta added this to the 3.0.2 milestone Aug 22, 2025
@DariuszDepta DariuszDepta self-assigned this Aug 22, 2025
@DariuszDepta DariuszDepta added the maintenance Maintenance label Aug 22, 2025
@DariuszDepta DariuszDepta marked this pull request as draft August 22, 2025 14:26
@DariuszDepta DariuszDepta changed the title Unified dependencies [WIP] Unified dependencies Aug 22, 2025
@DariuszDepta DariuszDepta requested a review from pinosu August 22, 2025 14:26
@DariuszDepta DariuszDepta marked this pull request as ready for review August 22, 2025 16:25
@DariuszDepta DariuszDepta changed the title [WIP] Unified dependencies Unified dependencies Aug 22, 2025
Copy link

@pinosu pinosu left a comment

Choose a reason for hiding this comment

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

Really nice! LGTM 💯

@DariuszDepta DariuszDepta merged commit fcfbf7e into main Aug 25, 2025
@DariuszDepta DariuszDepta deleted the unified-dependencies branch August 25, 2025 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants