Add travis-ci support by gesellix · Pull Request #92 · docker-java/docker-java
Navigation Menu
{{ message }}
docker-java / docker-java Public
- Notifications You must be signed in to change notification settings
- Fork 1.1k
Merged
docker-java-maintain merged 1 commit intodocker-java:masterfrom
gesellix-docker:ServiceLoaderClassLoader
Nov 13, 2014Merged
docker-java-maintain merged 1 commit intodocker-java:masterfrom
gesellix-docker:ServiceLoaderClassLoader
docker-java-maintain merged 1 commit intodocker-java:masterfrom
gesellix-docker:ServiceLoaderClassLoader
Conversation
Copy link
Contributor
gesellix
commented
Nov 13, 2014
gesellix
commented
This makes the DockerClientBuilder compile again (see #90 ). I also added a commit to configure travis-ci, so that in future pull requests basic correctness will be checked automatically (see #91). You still need to enable travis-ci to watch your repository, though.
Copy link
Contributor Author
gesellix
commented
Nov 13, 2014
gesellix commented
Nov 13, 2014@marcuslinke you have been faster :)
so, this now only contains the travis-ci config.
gesellix
changed the title
Use ClassLoader instead of ClassPath
Add travis-ci support
docker-java-maintain added a commit that referenced this pull request
Nov 13, 2014Add travis-ci support
docker-java-maintain
merged commit
945e821
into
docker-java:master
Copy link
Contributor
docker-java-maintain
commented
Nov 13, 2014
docker-java-maintain commented
Nov 13, 2014@gesellix Thanks!
Copy link
Contributor
docker-java-maintain
commented
Nov 13, 2014
docker-java-maintain commented
Nov 13, 2014Unfortunately Travis CI doesn't support Java 1.6 so deployed SNAPSHOTS are compiled under Java 1.7. Don't know if this is a problem as pom.xml declares
<jdk.source>1.6</jdk.source>
<jdk.target>1.6</jdk.target>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants