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

Add a module for Jackson 2.13 and make the 212 module use Jackson 2.12 #307

Merged
merged 1 commit into from
Mar 7, 2022

Conversation

maxstreese
Copy link

No description provided.

@maxstreese
Copy link
Author

This aims to fix #306

@RobertoUa
Copy link

is this going to be merged?

@maxstreese
Copy link
Author

@zmccoy would you be able to take a look at this? I am unsure who the current maintainers of this project are since @travisbrown announced he would step back if I recall correctly? The reason for this PR is explained in #306. Let me know if I can assist in any way.

@zmccoy
Copy link
Member

zmccoy commented Mar 4, 2022

@maxstreese Hey, thanks for the PR and ping! @zarthross and myself are currently trying to keep things tidy. I'll give this a look!

Copy link
Member

@zmccoy zmccoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One thing to note is that if anyone has switched to this and is using new functionality in the 213 module this will end up removing that from their classpath. The remedy is of course to have them use the 213 module instead.

@zmccoy zmccoy merged commit 525cb6d into circe:master Mar 7, 2022
@tlazaro tlazaro mentioned this pull request Feb 8, 2023
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.

3 participants