-
Notifications
You must be signed in to change notification settings - Fork 75
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
Plans on updating slf4j
to 2.0
#686
Comments
@hamnis made this comment, but then retracted. Why? Seems like it could work as a separate artifact. |
In a short perspective, bringing a new artifact is worse than doing a new major release. At the moment, we have no plans for a new major release in log4cats (like changing architecture/doing non-binary compatible changes on the whole). So it'd be fine to continue updating several release series, as we do it regularly within Typelevel. |
Just to throw out one more option: we could schism the log4cats-slf4j integration from this repository, into its own repository. Then it can have independent versioning, and we can bump the major version there without bumping it for core. |
https://www.slf4j.org/faq.html#compatibility
Honestly to me that just suggests we should pin to 1.x and move on. Anyone who wants to use with 2.x should be able to do so (although may require some build hacking), since it did not break bincompat of the |
I haven't seen a compelling argument for libraries to publish for 2.x. Apps that upgrade their backends will evict 1.x to something binary compatible with us, and apps that haven't won't silently break. The worst thing I can imagine by pinning is that we might push SBT eviction warnings downstream. |
I haven't seen any ill effects of pinning this in multiple downstream libraries. Can we close this as resolved that we stick to 1.x? |
Here I propose to discuss plans on migrating slf4j to 2.0. Since this migration isn't binary compatible we can't do this within the current major series. So we pinned its updates for now.
Ross has dropped some thoughts about it:
So this is a good entry point to start the discussion.
The text was updated successfully, but these errors were encountered: