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

Rework the Jackson module as a Jackson Jr alternative #1409

Merged
merged 2 commits into from May 16, 2021

Conversation

ashleyfrieze
Copy link
Contributor

Resolves #1408

This is a lightweight version of Jackson which can be used for smaller-scale applications. Please feedback.

pom.xml Outdated
Comment on lines 82 to 83
<jackson.version>2.12.3</jackson.version>
<jackson.databind.version>2.12.3</jackson.databind.version>
Copy link
Member

Choose a reason for hiding this comment

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

FYI, but we have two other PRs open to correct this. I'm going to merge those first. When that happens, you'll probably get a conflict. I'll resolve it for you.

Copy link
Member

@kdavisk6 kdavisk6 left a comment

Choose a reason for hiding this comment

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

@ashleyfrieze

Thank you for this contribution. I've approved this PR and marked it ready to merge, to allow other contributors to review. If there are no additional comments in a few days, this PR will be merged.

@kdavisk6 kdavisk6 added the ready to merge Will be merged if no other member ask for changes label May 15, 2021
@ashleyfrieze
Copy link
Contributor Author

@kdavisk6 That's brilliant. Thanks for letting me participate in this excellent project.

@velo velo merged commit 07850fc into OpenFeign:master May 16, 2021
@ashleyfrieze ashleyfrieze deleted the add-jackson-jr branch May 17, 2021 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Will be merged if no other member ask for changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Support for Jackson JR
3 participants