feat: Deprecate and remove scm commit title configuration

[josill]

Fix #1699

Remove the title field from SCM commit message configuration. Commit titles now always use the target name (or description as fallback), removing the ability to override the commit title.

Breaking Change: Configurations using commitMessage.title will need to remove that field. Ensure targets have meaningful name fields (which should already be the case).

Test

To test this pull request, you can run the following commands:

cd pkg/plugins/scms/git/commit
go test

Tradeoff

Removing the title configuration field reduces flexibility but simplifies the API and enforces consistency. Users who relied on custom commit titles will need to ensure their target names are descriptive enough for commit messages.

Potential improvement

Consider adding validation to ensure target names are always set and meaningful, since they now directly impact commit messages.

[olblak]

Thanks for the pull request, I still have it on my radar to review, but I didn't have the time yet

[olblak]

Consider adding validation to ensure target names are always set and meaningful, since they now directly impact commit messages.

Already the case, the pipeline will fail if the target name is not specified
cfr

updatecli/pkg/core/pipeline/target/main.go

Line 85 in 4027f0c

if t.Config.Name == "" {

Something we could improve (in a different pullrequest) is to tell jsonschema to require the name on

updatecli/pkg/core/pipeline/resource/main.go

Line 70 in 4027f0c

Name string `yaml:",omitempty"`

to provide a hint