-
Notifications
You must be signed in to change notification settings - Fork 29
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
Require Jenkins 2.361.4 or newer and Java 11 #23
Conversation
Subzer0570
commented
Jan 2, 2023
•
edited
Loading
edited
- Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
- Ensure that the pull request title represents the desired changelog entry
- Please describe what you did
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.
Thanks for being involved! More changes are needed.
The Jenkinsfile will need to be updated to remove the testing of Java 8, since this change will drop support for Java 8 and will require Java 11 or Java 17.
pom.xml
Outdated
<!-- <jenkins.version>2.346.3</jenkins.version>--> | ||
<jenkins.version>2.361.1</jenkins.version> |
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.
This change also needs the changes described in "Update Jenkins version". The bom entry will need to change form bom-2.346.x
to bom-2.361.x
. Based on "Update Jenkins version" it should require 2.361.4 rather than 2.361.1.
<!-- <jenkins.version>2.346.3</jenkins.version>--> | |
<jenkins.version>2.361.1</jenkins.version> | |
<jenkins.version>2.361.4</jenkins.version> |
@Subzer0570 this supersedes #21 and provides a better description for users. Thanks for doing it! |
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.
Looks good. I'd prefer to have the new comment removed from the bom section and would prefer to use the most recent version of bom-2.361.x
The Jenkinsfile will need to be updated to remove the testing of Java 8, since this change will drop support for Java 8 and will require Java 11 or Java 17. I am an administrator of ci.jenkins.io so I was able to modify the job definition with a replay. The check job should pass on the current run because I removed the Java 8 test from the job definition in the replay. |