Skip to content

Support configureForJackson2() with XmlMapper - #731

Merged
cowtowncoder merged 6 commits into
FasterXML:masterfrom
pjfanning:jackson2-config
Mar 7, 2025
Merged

cowtowncoder merged 6 commits into
FasterXML:masterfrom
pjfanning:jackson2-config

Conversation

@pjfanning

@pjfanning pjfanning commented Mar 6, 2025 •

Copy link
Copy Markdown
Member

@cowtowncoder will you notify me when the config changes are finished so that I can update this PR

@pjfanning
pjfanning marked this pull request as draft March 6, 2025 09:16
@cowtowncoder cowtowncoder changed the title support configureForJackson2 Support configureForJackson2() with XmlMapper Mar 6, 2025
@pjfanning
pjfanning marked this pull request as ready for review March 6, 2025 23:44
@cowtowncoder

Copy link
Copy Markdown
Member

@pjfanning Yes; realized I hadn't checked XmlReadFeature: changing now as #732, will add a note once merged.

@cowtowncoder

Copy link
Copy Markdown
Member

Ok, #732 now merged to master.

@cowtowncoder cowtowncoder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM: need just that one change wrt #732.

@pjfanning

Copy link
Copy Markdown
Member Author

LGTM: need just that one change wrt #732.

That should be supported now

@cowtowncoder
cowtowncoder merged commit 8a04feb into FasterXML:master Mar 7, 2025
cowtowncoder added a commit that referenced this pull request Mar 7, 2025
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