diff options
| author |   Francis Toth <francis.toth@yoppworks.com> | 2020-05-03 10:01:47 -0400 |
|---|---|---|
| committer |   Ofir Sonsino <ofir.sonsino@intl.att.com> | 2020-05-05 08:14:59 +0000 |
| commit | 762a1a320d92ca7415265066dbe4dfd7df91c2ec (patch) | |
| tree | 13abf9ad9948dca73403ba4720920b26ed15477b /common-app-api | |
| parent | 8394fae8e6fca37ad8f3e10ade82d8d2507585d1 (diff) | |
Refactor CsarUtils::collectComponentTypeArtifacts
This commit is a minor refactoring required to refactor CsarUtils::collectComponentInstanceArtifacts. In order to keep this commit small, we only focused CsarUtils::collectComponentTypeArtifacts, and removed any output that is not reflected by the function's result type. Modifying an argument is a actually a bad practice as explained in Clean Code by Robert Martin (cf. "Output arguments should be avoided"). This commit aims to enforce this aspect.
Change-Id: Ice4d6c9a78e7706c639dd60bc272253e298bc7be
Signed-off-by: Francis Toth <francis.toth@yoppworks.com>
Issue-ID: SDC-2812
Diffstat (limited to 'common-app-api')
0 files changed, 0 insertions, 0 deletions