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
chore(rust): remove unused dependencies #4475
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Terraform Cloud Plan Output
|
boringtun = { workspace = true } | ||
firezone-relay = { workspace = true } |
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.
Not sure what that was doing in here 😅
Performance Test ResultsTCP
UDP
|
80487f3
to
2d58339
Compare
2d58339
to
1684a40
Compare
fdfc23b
to
d615391
Compare
4a738af
to
1b3947f
Compare
I had to fix some warnings in our code (1b3947f). Those showed up (and failed the build) because I could move those to a separate PR but I am not sure it is worth it. |
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.
LGTM, I figure if you remove all that and it still compiles it's fine.
@@ -46,7 +42,7 @@ tauri = { version = "1.6", features = [ "dialog", "shell-open-api", "system-tray | |||
tauri-runtime = "0.14.2" | |||
tauri-utils = "1.5.3" | |||
thiserror = { version = "1.0", default-features = false } | |||
tokio = { version = "1.36.0", features = ["signal", "time"] } | |||
tokio = { version = "1.36.0", features = ["signal", "time", "macros", "rt", "rt-multi-thread"] } |
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.
Neat, it added those features that were accidentally pulled in from other packages in the workspace. That's been bugging me.
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 had to add those manually :)
#[derive(thiserror::Error, Debug)] | ||
pub(crate) enum Error {} |
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 surprised this one didn't show up in Clippy.
Thanks for taking over this one! |
These were all found by
cargo-udeps
.Resolves: #4403.