helm - remove trailing whitespaces (#337)

[alinabuzachis]

• Add remove_trailing_spaces() to remove trailing whitespaces for each helm_repository parameter

SUMMARY

ISSUE TYPE

• Bugfix Pull Request
• Docs Pull Request
• Feature Pull Request
• New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

[codecov[bot]]

Codecov Report

Merging #345 (dee55a3) into main (a9b8cc6) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #345   +/-   ##
=======================================
  Coverage   36.93%   36.93%           
=======================================
  Files           3        3           
  Lines         750      750           
  Branches      146      146           
=======================================
  Hits          277      277           
  Misses        424      424           
  Partials       49       49           

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a9b8cc6...dee55a3. Read the comment docs.

[goneri]

Hi @alinabuzachis, Does this PR fix an existing opened issue?

[tima]

@goneri I has some reservations about this one. Isn't this breaking from convention to how Engine works? The rest of Ansible engine doesn't strip whitespace like this. We should be consistent with that and not introduce something that breaks with that convention because one person made a typo.

[goneri]

Sorry, I understand. @tima I just pushed a PR to revert the patch.