-
Notifications
You must be signed in to change notification settings - Fork 40
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
Bump Jackson to 2.13.0 #91
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -68,7 +68,7 @@ | |
<changelist>-SNAPSHOT</changelist> | ||
<java.level>8</java.level> | ||
<jenkins.version>2.222.4</jenkins.version> | ||
<jackson.version>2.12.4</jackson.version> | ||
<jackson.version>2.13.0</jackson.version> | ||
<jackson-databind.version>${jackson.version}</jackson-databind.version> | ||
</properties> | ||
|
||
|
@@ -94,6 +94,11 @@ | |
<scope>import</scope> | ||
<type>pom</type> | ||
</dependency> | ||
<dependency> | ||
<groupId>jakarta.activation</groupId> | ||
<artifactId>jakarta.activation-api</artifactId> | ||
<version>1.2.2</version> | ||
</dependency> | ||
Comment on lines
+97
to
+101
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. jenkins ships with a lesser version (of jakarta.activation which itself bundles the same classes as the API 😱 ). if this is bundled in the plugin it is never used as the plugin is not using a plugin first classloader, so the fix should be to exclude the version instead. |
||
</dependencies> | ||
</dependencyManagement> | ||
|
||
|
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 also change
revision
to keep in step (not sure why we do not set that itself to$jackon.version
but likely it does not work when it comes to release time.