InseeFr / InseeFr/Eno

[Eno 3] Change mappers signature

Open
#572 0 comments 0 reactions 0 assignees View on GitHub
implementation discussion
Dominant language
Java
Stars
17
Forks
26
PR merge metrics
No merged PRs in 30d

Description

Currently, mapping methods have a signature like `void mapSomething(ObjectA objectHoldingValue, ObjectB emptyObjectCorresponding)`.

I think it would beneficial to refactor like this: `ObjectB mapSomething(ObjectA objectHoldingValue)`.

If it is possible (I might have missed practical reasons that would make this hard to do), this would make mapping methods/annotations more easily testable.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.