eclipse-sisu / sisu-project

Sisu Inject
https://eclipse.dev/sisu/
Eclipse Public License 2.0
18 stars 15 forks source link

Align SLF4J version with sisu.plexus #2

Closed cstamas closed 2 years ago

cstamas commented 3 years ago

Was 1.6.4 here vs 1.6.6 in sisu.plexus, went with higher.

mcculls commented 3 years ago

This particular dependency isn't actually used in the tests because the one from the PDE target platform takes precedence.

I'm currently updating the build to use Tycho 2.x which will let us use Maven dependencies in the PDE target platform, and avoids having them declared in different files.

mcculls commented 2 years ago

SLF4J 1.6.4 is now used consistently across modules