AxonFramework / cdi

Axon Framework CDI Support
Apache License 2.0
11 stars 10 forks source link

slf4j implementation conflict #18

Closed hwellmann closed 6 years ago

hwellmann commented 6 years ago

slf4j-simple should have test scope, not runtime scope.

A Maven artifact to be used as a library should not depend on any slf4j implementation with runtime or compile scope. This is likely to cause conflicts with any slf4j dependencies defined by the user of this library.

m-reza-rahman commented 6 years ago

Allard will review this question and provide his opinion. We want to be consistent with whatever Axon Framework proper does in this regard.

abuijze commented 6 years ago

I agree. Only the API should have compile scope dependency. The other slf4j dependencies should have test scope. This is also how it's done in Axon Framework itself.

m-reza-rahman commented 6 years ago

Resolved in 3.3-alpha1.