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

Travis CI Builds #21

Merged
merged 3 commits into from
May 25, 2016
Merged

Travis CI Builds #21

merged 3 commits into from
May 25, 2016

Conversation

samtstern
Copy link
Contributor

@kroikie please review, and if you choose to merge please use the Gthub 'squash' option.

This does the following:

  • Standardize build tools version, ignoring lint, and proguarding release variant
  • Add Travis CI configuration:
    • On pull requests, do a quick assembleDebug for all folders (~10 min on Travis)
    • On merged commits, build the whole thing (~30 min on Travis)

Future work:

  • Make the build faster so that we don't ever hit Travis' 50 minute limit.

  * Standardize build tools version
  * Add travis script
  * Make all samples not abort on lint and proguard their release builds
@kroikie
Copy link
Contributor

kroikie commented May 25, 2016

@samtstern could we travis encrypt the google-services.json file?

@samtstern
Copy link
Contributor Author

It's all just mock values I made up by smashing the keyboard. The
important thing is they pass basic regex and match the package name so the
build can succeed.

On Wed, May 25, 2016, 12:56 PM Arthur Thompson notifications@github.com
wrote:

@samtstern https://github.com/samtstern could we travis encrypt the
google-services.json file?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#21 (comment)

@kroikie
Copy link
Contributor

kroikie commented May 25, 2016

Ah so no real API keys, gotcha!

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.

2 participants