I guess we need some spent some words in documentation as well.

I checked the docs and we have nothing about this type of update methods. We have something in MappingTarget that only provides an example for "Update exist bean and return it".

I'll add something to the release notes though.

I checked the docs and we have nothing about this type of update methods. We have something in MappingTarget that only provides an example for "Update exist bean and return it".

I'll add something to the release notes though.

That's fine.. But should we mention these kind of methods in the documentation.? (It's more a completeness question)

That's fine.. But should we mention these kind of methods in the documentation.? (It's more a completeness question)

Seems I was wrong. We do have something in the docs:

Instead of void you may also set the method's return type to the type of the target parameter, which will cause the generated implementation to update the passed mapping target and return it as well

I think that this is fine and that we do not need any change, since what we say in the docs is what we are doing now as well.