swagger-coverage icon indicating copy to clipboard operation
swagger-coverage copied to clipboard

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

Open u3r opened this issue 3 years ago • 0 comments

I'm submitting a ...

  • [x] 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.

u3r avatar Jul 27 '22 10:07 u3r