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

update example #360

Merged
merged 3 commits into from
Apr 11, 2019
Merged

update example #360

merged 3 commits into from
Apr 11, 2019

Conversation

atecce
Copy link
Contributor

@atecce atecce commented Feb 20, 2019

i'm trying to run the example and i think it's a tad out of date. the only thing i've verified so far is this actually runs and listens on :9091, but i figured i'd just put it out there early and maybe update this as i play with it more.

@johanbrandhorst
Copy link
Contributor

Thanks a lot for this @atecce it does indeed look like we could do with these fixes. Assigning to @jonny-improbable as he's a bit more of a TS wiz than I!

Copy link
Contributor

@jonny-improbable jonny-improbable left a comment

Choose a reason for hiding this comment

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

LGTM, sorry for the slow turn-around.

@easyCZ easyCZ merged commit d998c3c into improbable-eng:master Apr 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants