Throw exception on type error in JsonDeserializer#8291
Conversation
|
As mentioned live, there is an attribute Looking around the codebase it's already in use in a couple of places, so I guess it has to be supported. |
|
Thanks for the PR! LGTM - but it seems this breaks one of the existing tests. |
|
That's caused by the type check during json deserialization. |
|
Ill continue with this once I pick up the serialization work again next week. |
fix int json deserialize check
|
@taniabogatsch This also now fixes the ugly |
|
Ah, that's a neat solution! |
Found by @Wang-Ji20 in #6919