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.

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

Bump Jackson version to la(te)st 2.13.x, 2.13.5 #1989

Open
wants to merge 1 commit into
base: 4.x
Choose a base branch
from

Conversation

cowtowncoder
Copy link

@cowtowncoder cowtowncoder commented Nov 16, 2024

Would increase version further but something in the build has pretty strict check wrt binary compatibility (need to figure out how to increase baseline).

EDIT: Looks like it's org.revapi that compares compatibility, and config file would be core/revapi.json

@CtrlAltFly
Copy link

@cowtowncoder - any reason jackson is still in 2.13X versions when 2.18 is out for sometime

@cowtowncoder
Copy link
Author

@CtrlAltFly As per my note, making a bigger jump requires tons of work updating revapi configs and I am not familiar enough to do that. So I ended up submitting just a trivial patch.

I don't know of any other blocker; but if someone tried resolving build/validation problem, I'd be happy to help with any Jackson compatibility issues there might be.

@CtrlAltFly
Copy link

@cowtowncoder - i agree. going several minor versions can break things. this patch (for now) seems to be a good one to be added for release.

so a 👍🏼 for your PR to be merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants