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

Update several dependencies to allow running on Java 11 (and also 8) #240

Merged
merged 4 commits into from
Aug 29, 2019

Conversation

balhoff
Copy link
Member

@balhoff balhoff commented Aug 29, 2019

No description provided.

@balhoff balhoff requested a review from goodb August 29, 2019 19:45
@kltm
Copy link
Member

kltm commented Aug 29, 2019

Just noting here that while a lot of our machines have access to Java 8 and Java 11 at need, some may not yet have a Java 11 available.

@balhoff
Copy link
Member Author

balhoff commented Aug 29, 2019

This is mostly to be able to test on java 11, not yet to request a move away from java 8.

@goodb
Copy link
Contributor

goodb commented Aug 29, 2019

Looks good to me. I tried it out here on mac laptop with java8 and everything is running smoothly including search api and recent additions for validation.

Does make me wonder if I should change coding environment from oracle java8 to openjdk11. @balhoff let me know if/when you think that is important.

@goodb goodb merged commit 3b278a9 into dev Aug 29, 2019
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.

3 participants