Tatu Saloranta
Tatu Saloranta
Although addition of a method in `DeserializationProblemHandler` is a viable option in general, handling happens at a point where it becomes bit unwieldy, and it's not clear if any other...
Ok, this is interesting but I need to digest it for a bit to know what I think. :) As to unit tests, they are under `yaml/src/test/java/...`.
Ok, no big objections, but I think we should figure out better name -- "Ext" would be ok for general-purpose extension, but here it is quite specific. I don't have...
Looks like code doesn't quite compile (refactoring?).
Thank you @HeikoBoettger-KarlStorz. One last thing (aside from my needing to re-review this) is CLA. Unless we've asked for it earlier (it only needs to be sent once & it's...
@HeikoBoettger-KarlStorz as per @yawkat's comment sending either one is enough. However, need to send `cla` NOT `clas` (seems to be common misunderstanding). Since I don't see the CLA could you...
CLA received. Also, ref to `[email protected]` fixed -- but I also did create alias so both `cla` and `clas` should actually work. Will need to review soon, bit overloaded but...
This looks good to me, but since I am leaving for an extended weekend, will not merge yet (partly since merge to `master`/3.0 may take some time) -- but will...
Merged in 2.19 for 2.19.0. NOTE: did NOT include in `master`/3.0 since merging was very difficult: partly due to Jackson side changes (which are sizable), but also since 3.0 uses...
Ok thank you for reporting the problem! Check is there for a reason so removing check would be dangerous (unless verifying it is not needed). I think to note is...