apache / logging-log4j-kotlin

A Kotlin-friendly interface to log against the Log4j API
https://logging.apache.org/log4j/kotlin
Apache License 2.0
45 stars 11 forks source link

Upgrade kotlin and log4j versions #27

Closed u-ways closed 1 year ago

u-ways commented 1 year ago

Hello,

It seems that the Log4j version of the Kotlin API is out of date. On top of that, the Kotlin version hasn't been updated for the past 2 years.

I have did a code analysis scan and found no incompatibles, tests are passing, and everything seems to be working as excepted.

Reasons to upgrade to 1.8.0 that are relevant:

And lots of other performance improvements overall...

Changes

Trivial changes - Feel free to amend as needed. :)

u-ways commented 1 year ago

@jvz Can you have a look please?

jvz commented 1 year ago

The Kotlin API is a minimum version requirement. I've been using this library with Kotlin 1.4.x, 1.7.x, and 1.8.x. The Log4j version update is good, though!

u-ways commented 1 year ago

I see, that makes sense, thanks for letting me know, I am closing this one and opening a more appropriate PR for Log4j upgrade only.