-
Notifications
You must be signed in to change notification settings - Fork 399
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 android example #1545
Add android example #1545
Conversation
8dd5a8f
to
6617732
Compare
This is mostly mirrored from https://github.com/keith/bazel-rust-mobile-demo but serves as a test / example to validate changes pushed to this repo. It requires you have an Android NDK / SDK setup to run.
6617732
to
ba48442
Compare
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.
Thank you so much! One question but otherwise looks awesome to me 😄
@@ -0,0 +1,50 @@ | |||
workspace(name = "android_examples") |
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.
Does this need to be it's own workspace? I feel we'd get better coverage by having it be a part of rules_rust_examples
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.
the only annoyance is if you want to run those locally, you have to have the android SDK / NDK setup which I imagine most testers won't, I'm happy to merge them if you want though!
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.
I think it can be done at a later date but in general, fewer workspaces would be better
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.
LGTM - thanks!
@keith You wouldn't happen to have an iOS example project lying around would you 😄 (Thanks again!) |
Thanks! iOS example here #1546 |
This is mostly mirrored from https://github.com/keith/bazel-rust-mobile-demo but serves as a test / example to validate changes pushed to this repo. It requires you have an Android NDK / SDK setup to run.