-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Enable DeserializationFeature.READ_ENUMS_USING_TO_STRING
by default (3.0)
#4566
Enable DeserializationFeature.READ_ENUMS_USING_TO_STRING
by default (3.0)
#4566
Conversation
Re-running CI because of Fuzzer failure (was nearly impossible to read log, due to its enormous volume) |
I do not recall seeing that error, but I also haven't been paying attention to Fuzzer action. I think I should just go ahead and disable it, seems to have little value and I think OSS-Fuzz runs their generation elsewhere anyway. Oddly enough, it runs only for... PRs? And one for |
…SING_TO_STRING-default-for-Jackson-3
@pjfanning pointed the obvious: Jackson package name means OSS-Fuzz code, built against 2.x, won't work. So this should now be merged, fixed. |
Will review this and #4567 later today, get merged. Good stuff! |
DeserializationFeature.READ_ENUMS_USING_TO_STRING
by defaultDeserializationFeature.READ_ENUMS_USING_TO_STRING
by default (3.0)
Will update JSTEP-2 Wiki accordingly. cc. @cowtowncoder |
I did some updates but may have missed something so appreciate your help @JooHyukKim. |
I checked and everything is there 👍🏼 |
Implements JSTEP-2 > DeserializationFeature