Skip to content

Commit

Permalink
use path specifiers for workspace deps in dev-dependencies (#587)
Browse files Browse the repository at this point in the history
See
https://github.com/MarcoIeni/release-plz/blob/main/crates/release_plz/Cargo.toml#L51-L53
for examples

Main point: if you have dev-dependencies, then you are compiling this
from a checkout, which means you don't need to resolve these from
crates.io

Our release automater, release-plz, doesn't order crates in
dev-dependencies before release, and we can resolve from path instead
for running benches
  • Loading branch information
a10y authored Aug 9, 2024
1 parent 7b790c5 commit 1892c75
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion encodings/zigzag/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ vortex-scalar = { workspace = true }
zigzag = { workspace = true }

[dev-dependencies]
vortex-fastlanes = { workspace = true }
vortex-fastlanes = { path = "../fastlanes" }

[lints]
workspace = true
6 changes: 3 additions & 3 deletions vortex-serde/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,9 @@ futures-executor = { workspace = true }
rand = { workspace = true }
simplelog = { workspace = true }
tokio = { workspace = true, features = ["full"] }
vortex-alp = { workspace = true }
vortex-fastlanes = { workspace = true }
vortex-sampling-compressor = { workspace = true }
vortex-alp = { path = "../encodings/alp" }
vortex-fastlanes = { path = "../encodings/fastlanes" }
vortex-sampling-compressor = { path = "../vortex-sampling-compressor" }

[lints]
workspace = true
Expand Down

0 comments on commit 1892c75

Please sign in to comment.