Skip to content
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

Merged
merged 1 commit into from
Apr 14, 2019

Conversation

thara
Copy link
Contributor

@thara thara commented Apr 7, 2019

This implements new functions below in Rust SDK and updates related documents and examples.

  • SetLabel(key, value)
  • SetAnnotation(key, value)
  • GameServer()
  • WatchGameServer(function(gameserver){…})

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.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 193007b7-d33e-4900-ae8b-56d76992c5b5

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

Copy link
Member

@markmandel markmandel left a 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)`.
Copy link
Member

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

thara added a commit to thara/agones that referenced this pull request Apr 14, 2019
This implements new functions below in Rust SDK and updates related documents and examples.

- SetLabel(key, value)
- SetAnnotation(key, value)
- GameServer()
- WatchGameServer(function(gameserver){...})
@thara
Copy link
Contributor Author

thara commented Apr 14, 2019

Thanks for your comment.
I fixed and force-pushed the branch for it.

@agones-bot
Copy link
Collaborator

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:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/693/head:pr_693 && git checkout pr_693
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.10.0-e5722ba

Copy link
Member

@markmandel markmandel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛴 awesome! Approved!

@markmandel markmandel merged commit bafcfea into googleforgames:master Apr 14, 2019
@markmandel markmandel added area/user-experience Pertaining to developers trying to use Agones, e.g. SDK, installation, etc kind/feature New features for Agones labels Apr 27, 2019
@markmandel markmandel added this to the 0.10.0 milestone Apr 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user-experience Pertaining to developers trying to use Agones, e.g. SDK, installation, etc kind/feature New features for Agones
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants