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 reflection-free Jackson deserialization to guide #46182

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rolfedh
Copy link
Contributor

@rolfedh rolfedh commented Feb 10, 2025

Add information about reflection-free Jackson deserialization to the guide.

Copy link

quarkus-bot bot commented Feb 10, 2025

Thanks for your pull request!

Your pull request does not follow our editorial rules. Could you have a look?

  • title should preferably start with an uppercase character (if it makes sense!)

This message is automatically generated by a bot.

@rolfedh
Copy link
Contributor Author

rolfedh commented Feb 10, 2025

Hi @mariofusco,
I've put together a rough draft for you to review.
I wrote this draft based on logical guesswork and assumptions rather than concrete facts, so I'd really appreciate your help in refining it for accuracy.
Thanks in advance!

This comment has been minimized.

Copy link

github-actions bot commented Feb 10, 2025

😭 Deploy PR Preview failed.

@rolfedh rolfedh force-pushed the add-jackson-deserializer-info branch from b704731 to 459972d Compare February 10, 2025 20:29

This comment has been minimized.

@rolfedh rolfedh force-pushed the add-jackson-deserializer-info branch from 459972d to 9a13bf4 Compare February 13, 2025 13:59
@rolfedh rolfedh changed the title [WIP] Add reflection-free Jackson deserialization to guide Add reflection-free Jackson deserialization to guide Feb 13, 2025
@rolfedh rolfedh requested a review from mariofusco February 13, 2025 14:01

This comment has been minimized.

@rolfedh rolfedh marked this pull request as draft February 13, 2025 19:20
@rolfedh
Copy link
Contributor Author

rolfedh commented Feb 13, 2025

@mariofusco We'll update this documentation after the release note is finalized.

@rolfedh rolfedh force-pushed the add-jackson-deserializer-info branch from 9a13bf4 to 341eb53 Compare February 14, 2025 14:40
@rolfedh rolfedh requested a review from mariofusco February 14, 2025 14:40
@rolfedh rolfedh marked this pull request as ready for review February 14, 2025 14:40
@rolfedh
Copy link
Contributor Author

rolfedh commented Feb 14, 2025

Hi @mariofusco,

I've updated the community docs to align with the release note you approved. Thanks for your review!

Copy link

quarkus-bot bot commented Feb 14, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 341eb53.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

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

Successfully merging this pull request may close these issues.

2 participants