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

add remaining rlbot interface functions + one example #3

Merged
merged 1 commit into from
Sep 25, 2018

Conversation

ehsanul
Copy link
Contributor

@ehsanul ehsanul commented Sep 19, 2018

No description provided.

src/rlbot_generated.rs Outdated Show resolved Hide resolved
src/rlbot.rs Outdated Show resolved Hide resolved
@ehsanul ehsanul force-pushed the flesh_out_rlbot_interface branch 2 times, most recently from 8ddc4ae to 7b1ebeb Compare September 20, 2018 00:58
Copy link
Member

@whatisaphone whatisaphone left a comment

Choose a reason for hiding this comment

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

Looks good for the most part, and I really like the new example!

Sorry it took me so long to review this

examples/gravity.rs Outdated Show resolved Hide resolved
examples/simple_flatbuffer.rs Outdated Show resolved Hide resolved
examples/gravity.rs Show resolved Hide resolved
examples/gravity.rs Outdated Show resolved Hide resolved
examples/simple_flatbuffer.rs Show resolved Hide resolved
src/rlbot.rs Outdated Show resolved Hide resolved
src/rlbot.rs Outdated Show resolved Hide resolved
src/rlbot.rs Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
src/dll.rs Outdated
pub set_game_state: SetGameState,
pub render_group: RenderGroup,
pub send_chat: SendChat,
/// flatbuffer version of send_chat
Copy link
Member

Choose a reason for hiding this comment

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

nit: Can you capitalize this properly? (even though it's not public)

@ehsanul ehsanul force-pushed the flesh_out_rlbot_interface branch 3 times, most recently from d235d8e to bcf62c5 Compare September 25, 2018 06:10
@ehsanul
Copy link
Contributor Author

ehsanul commented Sep 25, 2018

@whatisaphone Thanks for reviewing. I've made your suggested changes, should be good to go now!

@whatisaphone whatisaphone merged commit ca42021 into RLBot:master Sep 25, 2018
@whatisaphone
Copy link
Member

Looks good, thanks!

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.

None yet

2 participants