viclovsky/swagger-coverage

swagger-coverage-commons should not depend on logback (only slf4j)

u3r opened this issue · 0 comments

u3r commented

I'm submitting a ...

  • bug report
  • feature request

What is the current behavior?

swagger-coverage-commons has a direct dependency implementation("ch.qos.logback:logback-classic") on logback

If the current behavior is a bug, please provide steps to reproduce, broken swagger specification and swagger-coverage-output:

build and check transitive dependencies when importing.

What is the expected behavior?

swagger-coverage-commons should only have a transitive dependency on slf4j.
If needed for testing, replace with testImplementation("ch.qos.logback:logback-classic") otherwise just drop this dependency (and rely on dependent code (e.g. CLI) to declare the actual implementation.

What is the motivation / use case for changing the behavior?

If not implemented all the benefit of SLF4J is wasted.