Skip to content

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
velo added a commit that referenced this pull request Oct 7, 2024
velo added a commit that referenced this pull request Oct 8, 2024
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