AppHubPlatform / apphub

apphub is an iOS client for AppHub
https://apphub.io
Other
206 stars 29 forks source link

[Android] Add android tests to circle.yml #16

Open arbesfeld opened 8 years ago

mccjul commented 8 years ago

Hey so i've forked this repo to add tests in circe but I've had success with travis when I isolate the android repo. I contacted circle and they said that you can't do android build on an apple os. This was my travis.yml

language: android
jdk: oraclejdk8
android:
  components:
    - build-tools-22.0.1
    - android-22
    - add-on
    - extra

script: 
  - ./gradlew test --info

However if you don't isolate the repo and try to cd into android it doesn't work.

GeoffreyPlitt commented 8 years ago

Does apphub actually work for android?

mccjul commented 8 years ago

I don't want to answer for the guy that actuaIIy made this. The service doesn't have android support yet. However, the android branch of this repo is something to Iook at. With a bit of Iove, it worked on my end.

GeoffreyPlitt commented 8 years ago

Thanks @mccjul! Can you elaborate on the "love" required?

@arbesfeld What's the ETA for officially supporting android, in a supported way?

arbesfeld commented 8 years ago

@mccjul I would be happy to review your changes and get them merged into master. I'm curious, are you using the AppHub hosting or running your own build server?

@GeoffreyPlitt we've found most people are happy with the Google Play updating mechanisms, but we'd be happy to consider Android if enough people are interested.

GeoffreyPlitt commented 8 years ago

@arbesfeld Cool. Yes, we'd like to use a single mechanism that works for updating everywhere, so we're hoping to see the Android side!

mccjul commented 8 years ago

I separated it into a different repository IocaIIy to onIy have android to get the tests to work and then mess with the zipping tiII it worked on my end. However when I put it with the rest of the Library... things wouId get hairy. Which is why I haven't done a puII request yet. I'm working on it though.