-
-
Notifications
You must be signed in to change notification settings - Fork 331
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 requireTypeIdForSubtypes
property for JsonTypeInfo.Value
in Jackson 3.0
#228
Add requireTypeIdForSubtypes
property for JsonTypeInfo.Value
in Jackson 3.0
#228
Conversation
======= | ||
* | ||
* @since 2.3 but databind <b>only since 2.5</b>. | ||
>>>>>>> 2.16 | ||
>>>>>>> 2.16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a whitespace just to highlight this. I couldn't decide whether to remove @since...
part or keep 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmh. My mistake, shouldn't have left these around. For 3.0 all "@SInCE 2.x" notes to be removed (assumption being that anything in 3.0 or earlier does not need it).... although now wondering if same should apply to annotations package or not.
But for now, yeah let's remove all @since
s (except maybe in some places can add @since 3.0
for something introduced only in 3.x.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ok after merging the other PR, need to resolve conflict here. |
…rty-for-JsonTypeInfo.Value-in-3.0
relates to #226