-
-
Notifications
You must be signed in to change notification settings - Fork 3k
WIP convert to jackson 3 #14259
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
base: main
Are you sure you want to change the base?
WIP convert to jackson 3 #14259
Changes from all commits
5c07d88
511f42a
f5f6682
2ab4cda
9649cc3
1a2979b
da9133e
c07d71c
cc621ce
b9fe2e4
398f1eb
2b33910
431c9e9
9b30927
bbce853
7c8f43d
f291512
395fade
ee56c1e
bc7b308
bf6b1dc
d7b6e80
b828262
a1ab067
b4aaae4
f404793
5aee419
fcfebab
5891f4b
00f1f31
62bdd16
99b55b7
f345a8f
ed5520f
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 |
|---|---|---|
|
|
@@ -32,6 +32,10 @@ dependencies { | |
| api("org.junit.jupiter:junit-jupiter") | ||
| api("org.junit.platform:junit-platform-launcher") | ||
|
|
||
| api(platform("tools.jackson:jackson-bom:3.0.2")) | ||
|
Member
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. The version has to go to below to the constraints (doesn't it?)
Member
Author
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. I have absolutely no idea, because it's a bom and other boms are declared at the top as well |
||
| api("tools.jackson.core:jackson-core") | ||
| api("tools.jackson.core:jackson-databind") | ||
| api("tools.jackson.dataformat:jackson-dataformat-yaml") | ||
| } | ||
|
|
||
| dependencies.constraints { | ||
|
|
@@ -47,8 +51,6 @@ dependencies.constraints { | |
| api("com.dlsc.gemsfx:gemsfx:3.6.2") | ||
| api("com.dlsc.pdfviewfx:pdfviewfx:3.3.2") | ||
| api("com.ibm.icu:icu4j:72.0.1!!") | ||
| api("com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:2.20.1") | ||
| api("com.fasterxml.jackson.datatype:jackson-datatype-jsr310:2.20.1") | ||
| api("com.fasterxml:aalto-xml:1.3.4") | ||
| api("org.eclipse.lsp4j:org.eclipse.lsp4j:0.24.0") | ||
| api("org.eclipse.lsp4j:org.eclipse.lsp4j.websocket:0.24.0") | ||
|
|
||
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.
Maybe, this can be left out, because it will come in using the jablib SNAPSHOT from maven central.