Skip to content
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.

Sign up for GitHub

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

Jump to bottom

SLF4J: Class path contains multiple SLF4J bindings #434

Closed
natros opened this issue Jul 11, 2020 · 5 comments · Fixed by #435
Closed

SLF4J: Class path contains multiple SLF4J bindings #434

natros opened this issue Jul 11, 2020 · 5 comments · Fixed by #435
Labels
done enhancement
Milestone
9.1.2

Comments

@natros
Copy link
Contributor

natros commented Jul 11, 2020

cron-utils should not provide any slf4j implementation like slf4j-simple to avoid multiple slf4j bindings in classpath
@jmrozanec
Copy link
Owner

@natros thank you for the insight. May we ask you for a PR with a fix? Will be glad to check and merge. Thanks! 😄

@natros natros mentioned this issue Jul 11, 2020
Merged
@jmrozanec jmrozanec added this to the next milestone Jul 13, 2020
@francisdb
Copy link
Contributor

@jmrozanec can this please be released?

@jmrozanec
Copy link
Owner

@francisdb sure. Will release today EOD. Thank you for pushing forward! 😄

@jmrozanec
Copy link
Owner

@francisdb a new version was released! We apologize for the delay! 😄

@francisdb
Copy link
Contributor

Thanks a lot, no need for apologies!

@jmrozanec jmrozanec added the done label Nov 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
done enhancement
Projects
None yet
Milestone
9.1.2
Development

Successfully merging a pull request may close this issue.

slf4j-simple is only needed for testing natros/cron-utils
3 participants
@francisdb @jmrozanec @natros