Configuration-as-code-plugin: Mapping#putIfNotEmpry

[aheritier]

Description

• [ ] Please mark the incorrectly spelled method putIfNotEmpry deprecated
• [ ] add new method called putIfNotEmpty
• [ ] old method should call new method (to avoid duplication)
• [ ] add a javadoc for old method to suggest use new method instead. (See howto)

The problem with IDEs is that everyone is using the autocompletion and doesn't see such misprint.

https://github.com/jenkinsci/configuration-as-code-plugin/blob/045631a8bf8b2fb0ac219128a82dca5b34716869/plugin/src/main/java/io/jenkins/plugins/casc/model/Mapping.java#L50

@ndeloof one more 🍺 for me :-)

[jetersen]

Good candidate for hacktoberfest to fix the spelling.

[bojanfilipovic]

Hi! Can I pick this up as part of the hacktoberfest?

[jetersen]

@bojanfilipovic Please go ahead, please check the CONTRIBUTING