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.
Hi folks! What do you think about adding kotlin-reflect as a dependency?
I have a custom plugin that needs jackson-module-kotlin, which in turn requires kotlin-reflect. Today I can build up this dependency tree below, but unfortunately I get a
kotlin.jvm.KotlinReflectionNotSupportedError: Kotlin reflection implementation is not found at runtime. Make sure you have kotlin-reflect.jar in the classpath
By adding kotlin-reflect to okhttp-api, the exception goes away. The dependency tree now looks like:
This works great for me and I think is a practical solution, but I can also see how it may be better to split off a kotlin-api plugin instead. Happy to do that if it's preferred. For what it's worth, my only uses for kotlin involve okhttp and jackson.
Testing done
Published my version and ran with it locally, proving the runtime exception no longer happens.
Existing tests pass.
Submitter checklist