-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Issue #3051 - New Simplified Jenkinsfile #3052
Issue #3051 - New Simplified Jenkinsfile #3052
Conversation
I want to fix issue #2458 after this (removing jdk9 and jdk10 builds from CI) |
Referencing Issue #3051 (typo in original commit) |
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
9b5e0f6
to
7b8bda8
Compare
// in case of change update method isMainBuild | ||
def jdks = ["jdk8","jdk9","jdk10","jdk11"] | ||
def mainJdk = "jdk8" | ||
def jdks = [mainJdk, "jdk9", "jdk10", "jdk11"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jdk9 and jdk10 are not removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not yet in jetty-9.4.x (maybe could make sense? I don't really have strong opinion)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing jdk9 and jdk10 will be a different PR/commit (it's a different issue: #2458)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
// in case of change update method isMainBuild | ||
def jdks = ["jdk8","jdk9","jdk10","jdk11"] | ||
def mainJdk = "jdk8" | ||
def jdks = [mainJdk, "jdk9", "jdk10", "jdk11"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not yet in jetty-9.4.x (maybe could make sense? I don't really have strong opinion)
Signed-off-by: Joakim Erdfelt joakim.erdfelt@gmail.com