-
Notifications
You must be signed in to change notification settings - Fork 810
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
Implement lacking functions in Rust SDK #693
Conversation
Build Failed 😱 Build Id: 193007b7-d33e-4900-ae8b-56d76992c5b5 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
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.
Thanks for submitting this! 🍰
A couple of small things to fix, and this should be good to go.
- I made a note about the feature shortcode below
- If you rebase this against master, what is currently breaking in tests will pass
- Please squash this down to a single commit as per Contributor Guidelines
Once that's done - happy to approve 👍
@@ -58,3 +53,37 @@ if sdk.shutdown().is_err() { | |||
println!("Could not run Shutdown"); | |||
} | |||
``` | |||
|
|||
To [set a Label]({{< relref "_index.md#setlabel-key-value" >}}) on the backing `GameServer` call `sdk.set_label(key, value)`. |
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.
Small thing, this will need to be wrapped in a feature shortcode for "0.10.0" so it's hidden until the next release.
For example:
https://agones.dev/site/docs/contribute/#within-a-page
This implements new functions below in Rust SDK and updates related documents and examples. - SetLabel(key, value) - SetAnnotation(key, value) - GameServer() - WatchGameServer(function(gameserver){...})
bca9542
to
e5722ba
Compare
Thanks for your comment. |
Build Succeeded 👏 Build Id: 039c5649-45e0-4ccd-8c68-188734161baf The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
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! Approved!
This implements new functions below in Rust SDK and updates related documents and examples.
I believe this PR make that the Rust SDK have all the SDK functionality.
This PR includes reformatting code by rustfmt.
If it is not acceptable, I would make another one including reformatting code only.