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

Bring up continuous integration for commits and pull requests #14

Merged
merged 5 commits into from
Feb 24, 2017
Merged

Conversation

jpd236
Copy link
Collaborator

@jpd236 jpd236 commented Feb 16, 2017

-Use more modern build tools (Java 1.8, Gradle 3.3, Gradle Plugin 2.2.3, v25 Build Tools and SDK)
-Make everything build using these tools
-Add a Travis CI configuration which builds code and Javadoc, lints, and runs unit tests

Travis CI is configured to run on any commit or pull request (including this one) as long as a .travis.yml is present. Once this pull request is merged, the restriction on .travis.yml being present will be lifted (just in case someone tries to put together a commit which removes it inadvertently).

See #1

-Move to more modern build tools and update scripts/code to work
-Add gradle wrapper rather than relying on system version
Per the previous commit, Volley itself will still compile using Java
1.7 source/target compatibility.
The built-in tools are not aware of the v25 SDK and can't install it.
@jpd236 jpd236 requested a review from yilanl February 16, 2017 21:56
@jpd236 jpd236 merged commit bc67181 into master Feb 24, 2017
@jpd236 jpd236 deleted the ci branch February 24, 2017 17:40
@wx2013 wx2013 mentioned this pull request May 3, 2017
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