-
Notifications
You must be signed in to change notification settings - Fork 364
Fix #673: Bring back Java 7 support #693
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
Conversation
|
I managed to get the build to pass on JDK 7 using |
|
LGTM - nice work! |
|
for some reason Travis which runs "mvn test -B" passes but trying to deploy which goes through "mvn verify" fails. I was trying to cut a release this morning. Can someone find out why some groovy tests are failing using "mvn verify" ? If not we'll have to revert this change. Common error seen: |
|
I guess |
|
Ok, gonna update travis to always test using "mvn verify" |
|
Yes. Sorry. Should have thought of that before. |
distributionmodule depends onopenstack4j-resteasy