-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cargo cleanup #667
Cargo cleanup #667
Conversation
We can almost eliminate serde, but there are some metrics packages using it, so it stays for now.
dc94e6b
to
36029d1
Compare
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.
There are some unused dependencies left. You can run cargo machete
to list them automatically, though there is a false positive for serde
and prost
.
diff --git a/grpc-testtool/Cargo.toml b/grpc-testtool/Cargo.toml
index f7a22fe..24843f6 100644
--- a/grpc-testtool/Cargo.toml
+++ b/grpc-testtool/Cargo.toml
@@ -18,12 +18,10 @@ bench = false
[dependencies]
firewood = { version = "0.0.4", path = "../firewood" }
prost = "0.12.3"
-thiserror = "1.0.57"
tokio = { version = "1.36.0", features = ["sync", "rt-multi-thread"] }
tonic = { version = "0.11.0", features = ["tls"] }
tracing = { version = "0.1.40" }
clap = { version = "4.5.0", features = ["derive"] }
-tempdir = "0.3.7"
log = "0.4.20"
env_logger = "0.11.2"
chrono = "0.4.34"
diff --git a/storage/Cargo.toml b/storage/Cargo.toml
index b4efac6..da7356e 100644
--- a/storage/Cargo.toml
+++ b/storage/Cargo.toml
@@ -13,8 +13,6 @@ hex = "0.4.3"
serde = { version = "1.0.199", features = ["derive"] }
smallvec = { version = "1.13.2", features = ["serde", "write", "union"] }
sha2 = "0.10.8"
-bytemuck = { version = "1.14.3", features = ["derive"] }
-integer-encoding = "4.0.0"
[dev-dependencies]
rand = "0.8.5"
Leaving serde alone since I think we may be able to remove it
Great suggestion, thanks! I left serde alone because I think we might be able to remove it completely later. |
@@ -13,7 +13,7 @@ hex = "0.4.3" | |||
serde = { version = "1.0.199", features = ["derive"] } | |||
smallvec = { version = "1.13.2", features = ["serde", "write", "union"] } | |||
sha2 = "0.10.8" | |||
bytemuck = { version = "1.14.3", features = ["derive"] } | |||
integer-encoding = "4.0.0" |
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.
Looks like you left this guy
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, we'll need it very soon as the one user (hashednode.rs) will be moving to storage very soon.
Removed some unused dependencies from the main project.