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

build(ci): build using a java 11 toolchain #4927

Merged
merged 4 commits into from
Nov 5, 2021
Merged

build(ci): build using a java 11 toolchain #4927

merged 4 commits into from
Nov 5, 2021

Conversation

keturn
Copy link
Member

@keturn keturn commented Oct 16, 2021

Note this is not yet changing the compatibility level. This is setting up the build environment to make sure it can use java 11 first.

As this changes only the Jenkinsfile, it has no impact on local development environments. Our guide for setting up a development workspace has already recommended using Java 11 for a while now.

Requires

  • build: use java 11 ModuleJteConfig#21
    Because modules depend on engine, I expect things will go smoother if the module-building environment is Java 11-enabled before we start publishing Java 11 engine builds.

Note this is not yet changing the compatibility level. This is setting up the build environment to make sure it can use java 11 first.
@github-actions github-actions bot added the Category: Build/CI Requests, Issues and Changes targeting gradle, groovy, Jenkins, etc. label Oct 16, 2021
@keturn
Copy link
Member Author

keturn commented Oct 16, 2021

Java Version

@keturn keturn merged commit 2e17054 into develop Nov 5, 2021
@keturn keturn deleted the build/java11 branch November 5, 2021 20:22
keturn added a commit that referenced this pull request Nov 5, 2021
@keturn keturn restored the build/java11 branch November 5, 2021 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Build/CI Requests, Issues and Changes targeting gradle, groovy, Jenkins, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants