search

OrienteerBAP / JNPM

Native Java API to work with JavaScript Node Package Manager (NPM): query, retrieve, pack into jar (webjars), CDN
Apache License 2.0
20 stars 1 forks source link

Update Jackson to get rid of deprecation messages #23

Closed lppedd closed 8 months ago

lppedd commented 8 months ago

Thanks for the library!
I'm using it in a custom Gradle task, and every execution I receive deprecation messages from Jackson:

PropertyNamingStrategy.SnakeCaseStrategy is used but it has been deprecated due to risk of deadlock. Consider using PropertyNamingStrategies.SnakeCaseStrategy instead. See https://github.com/FasterXML/jackson-databind/issues/2715 for more details.

It would be cool to update it, or remove usages of deprecated symbols.

If you don't have the time, let me know and I'll do it.

PhantomYdn commented 8 months ago

Makes perfect sense. Thank you! Are you fine with using SNAPSHOT version for some period time or prefer some intermediate minor version?

lppedd commented 8 months ago

@PhantomYdn up to you! What makes less work or more sense (as long as it ends up in Maven Central I guess).

For better context here is the usage: https://github.com/lppedd/kotlin-externals/blob/master/buildSrc/src/main/kotlin/com/github/lppedd/kotlin/gradle/services/NpmService.kt

PhantomYdn commented 8 months ago

@lppedd , just pushed changes to github and mvn central as SNAPSHOT Thank you for awaiting!

lppedd commented 8 months ago

@PhantomYdn thanks! I can confirm it works ok without any deprecation message now.