-
Notifications
You must be signed in to change notification settings - Fork 66
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 POM for next release #87
Conversation
<configuration> | ||
<rules> | ||
<enforceBytecodeVersion> | ||
<excludes combine.children="append"> |
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.
No longer necessary after jenkinsci/plugin-pom#136 (confirmed that on parent pom 3.28 the config is necessary, but on 3.29+ it is not).
<changelist>-SNAPSHOT</changelist> | ||
<jenkins.version>2.121.1</jenkins.version> | ||
<java.level>8</java.level> | ||
<plugin.minimumJavaVersion>11</plugin.minimumJavaVersion> |
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.
The code will work fine on Java 8 as well, and we want to release it to everyone.
@@ -32,7 +32,6 @@ | |||
import org.jenkinsci.plugins.workflow.job.WorkflowRun; | |||
import org.jenkinsci.plugins.workflow.test.steps.SemaphoreStep; | |||
import org.junit.ClassRule; | |||
import org.junit.Ignore; |
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.
I should have removed the import in #86.
@@ -42,7 +42,6 @@ | |||
import org.jenkinsci.plugins.workflow.test.steps.SemaphoreStep; | |||
import static org.junit.Assert.*; | |||
import org.junit.ClassRule; | |||
import org.junit.Ignore; |
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.
Should have been removed in #86.
Minor updates just to get things ready for the release (which will support Java 8 in addition to newer Java versions). Once this PR is merged, we should merge the
java11
branch intomaster
and release it.