Implicit option names should be kebabed

[SUPERCILEX]

Example:

@Option(description = "Trigger default templates update")
@get:Input
var updateTemplates: Boolean = false

The option name becomes updateTemplates but it should be update-templates.

[SUPERCILEX]

@eriwen WDYT? And how would we deprecate the old properties? Or maybe save this up for Gradle 6.0?

[eriwen]

This does seem like a breaking change (if minor), so in my opinion it should be Gradle 6.0. Possibly name should be required.

I'd say "a contribution is welcome" but it would be kinda weird to have to save a PR for nearly a year to merge it.

[SUPERCILEX]

Yeah, that's what I was thinking. 👍 Well, I'm already excited for Gradle 6! 😆

[big-guy]

We could have both options available and emit a deprecation warning for the non-kebab names.

We should update the docs to say that specifying an option value is a good practice for setters/fields that have names that should be kebabed. This is what we're doing in InitBuild.

[CristianGM]

This was waiting for Gradle 6.0 to do a small breaking change, now this train seems to be gone..

[stale]

This issue has been automatically marked as stale because it has not had recent activity. Given the limited bandwidth of the team, it will be automatically closed if no further activity occurs. If you're interested in how we try to keep the backlog in a healthy state, please read our blog post on how we refine our backlog. If you feel this is something you could contribute, please have a look at our Contributor Guide. Thank you for your contribution.

[Vampire]

Would still be nice to have, not present in 6.7.1.
7.0 is around the corner, so maybe now would be a good time for the breaking change? :-)