NordicSemiconductor / Kotlin-BLE-Library

BSD 3-Clause "New" or "Revised" License
314 stars 33 forks source link

Update the library to SLF4J 2.0 #140

Open LionZXY opened 5 months ago

LionZXY commented 5 months ago

Background

Right now the library is using <2 version of slf4j. This seems wrong to me, as the current version of slf4j is > 2.0.

So I have two questions:

philips77 commented 5 months ago

Hello,

If you'd release an updated version of SLF4J-Timber with updated SLF4J version, that would be amazing. I can migrate to use the latest version, but please also release it on Maven Central.

Why decided to use slf4j instead of the past self-described logging system?

We decided not to reinvent the wheel and use something already existing with more features. Also, I had to fix dependency issues and uncouple BLEK from the Nordic Common library, which should in theory only be used for our sample apps, without forcing people to use it, actually, if that's not required.

We've been already using SLF4J in nRF Connect Device Manager app and I liked how easily can be integrated with Timber or even released as non-android library.