-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove the slf4j-nop from compile classpath #46
Labels
Milestone
Comments
Hi @sureshg, thank you for pointing that out! |
firaja
added a commit
that referenced
this issue
Nov 15, 2021
Merged into master |
Thank you for making the change, is this going to be reflected in the maven repo sometime soon as new version? 1.5.4 or 1.6.0? Cheers |
Hi @firaja got your notification, much appreciated! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
You shouldn't add the
slf4j
nop
impl to the classpath ( https://github.com/Password4j/password4j/blob/master/pom.xml#L53) as it causing package conflict issues for modular applications. Usually application using this library brings proper slf4j impl.The text was updated successfully, but these errors were encountered: