-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
Add JDK 9 support #1188
Add JDK 9 support #1188
Conversation
@jimschubert I've updated the gradle version as well: https://github.com/OpenAPITools/openapi-generator/pull/1188/files#diff-100635b319e4bc29ba29da64405bcea4R57 Let me know if you want to keep the current version instead. |
Shouldn't we jump directly to the LTS JDK 11 ? |
@cbornet yup, we can consider that. This PR only makes the project buildable/compilable with JDK9. The compiler option is still set to 1.8 in pom.xml. |
* update jmockit version * list jvm * test jdk9 in appveyor * use latest version of gradle * update to jdk9 * use openjdk9 * use default java version in appveyor * use jdk8 docker image * consolidate bash, ios pom into parent pom * consolidate circlecii pom into parent pom * remove circleci pom.xml
PR checklist
./bin/
to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.sh
and./bin/security/{LANG}-petstore.sh
if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\
.master
,3.4.x
,4.0.x
. Default:master
.Description of the PR
jmockit
installation instruction to pom.xml (Ref: http://jmockit.github.io/tutorial/Introduction.html#runningTests)cc @OpenAPITools/generator-core-team