Eugene Lukash

Results 290 comments of Eugene Lukash
trafficstars

> I am not sure if JSR-303 validation fits at all, even with generated getters... Surely, we cannot expect that existing implementations of the bean validation will handle immutable object...

Hi @gunnarmorling, Thank you for chiming in! I've spent some time today experimenting with BV and I think we already pretty close to say that we support BV. I don't...

> The idea of "mixing in" validator support through some base interface is neat indeed (though it creates a dependency to an Immutables-defined interface at runtime, not sure whether there...

@brentryan totally agree with the concerns. Just don't know a reasonable way to deal with it without generating additional proxies (a la `ImmutableX.Json` for Jackson) and so on, which is...

Hi @jifwin. Currently, validation is supported as is: using built-in validator (`Style.validationMethod`) or by using mixin approach outlined above, is there specific aspect or fix you're interested in? Known issues...

I don't really understand how `testOnly` should work, however it seems that you can just create test instances and then use `Builder.from(testSample)` to fill all the default values before overriding...

I've reread this, but still I somehow feel the use case is easily addressed by simpler means. Code generation should not be used to sweep everything under the rug, it...

Hi Sebastian Thank you for the feature request! I guess we need to think about that. While I'm digesting variants described, it is interesting to know what kind of tracking...

That would definitely bring in some power definitely, good idea! However I feel that power to weight of pre/post hooks is lower than I would expect, given my imagination let...

I dunno. One one hand there's new, still experimental functionality to generate meta-model class for immutable objects via `@Data` annotation in `org.immutables:data`( https://repo1.maven.org/maven2/org/immutables/data/2.8.6.datatypes/ or https://repo1.maven.org/maven2/org/immutables/data/2.9.0-beta2/ ), not documented yet, so...