Bump mbknor-jackson-jsonschema to 2.13 #81
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bumping
mbknor-jackson-jsonschema
to2.13
.But, to expand on the issue:
Dependency in case:
As far I could see, this dependency is used to generate a json schema when using "advanced options" such as function calls.
For example, when using this together with Quarkus, it breaks because the dependency is build for Scala up to 2.13.
When running this with current dependency (
v 2.12
) it breaks and need to be removed and2.13
added manually.But, to make it worse, the dependency is quite old and probably not maintained anymore: last release was done more than 4y ago.
openai-java
, the official SDK for OpenAI was also released recently and they don't use this dependency.It might be worth checking their code and understanding how they have done it and improve this lib with that.