-
Notifications
You must be signed in to change notification settings - Fork 13
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
add a rust trycp client crate #209
Conversation
neonphog
commented
May 31, 2024
•
edited
Loading
edited
- Pulls some shared protocol types out into a trycp_api crate to be used by both the server and client.
- Adds a trycp_client crate that can be used to talk to a trycp server.
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 great!
crates/trycp_api/Cargo.toml
Outdated
@@ -0,0 +1,10 @@ | |||
[package] | |||
name = "trycp_api" | |||
version = "0.0.100-alpha0" |
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.
We should probably start versioning these, can it just match the tryorama minor version for now @jost-s?
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.
Cool
crates/trycp_client/Cargo.toml
Outdated
serde_json = "1.0.117" | ||
tokio = { version = "1.38.0", features = [ "full" ] } | ||
tokio-tungstenite = "0.21" | ||
trycp_api = { version = "0.0.100-alpha0", path = "../trycp_api" } |
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.
To the workspace for easy finding when updating versions? No release-automation here :)
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.
Awesome!
I suggest smoke tests for the client at some stage.
Yep, absolutely agree - for now Wind Tunnel is going to be using this so we'll have reasonably quick feedback if something breaks :) |