Allow commitMessageSuffix without an automatically inserted space
#45092
paduszyk
started this conversation in
Suggest an Idea
Replies: 1 comment
|
I also tried appending the period through {
"commitMessageExtra": "{{ commitMessageExtra }}."
}This does not work because |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Tell us more.
I would like to configure Renovate commit messages so that they end with punctuation, for example:
I tried:
{ "commitMessageAction": "Updated", "commitMessageSuffix": ".", "commitMessageLowerCase": "never" }However, this produces:
There is an unwanted space before the period.
This commit-message style is used by Django, whose commit messages conventionally begin with a capitalized past-tense verb and end with a period. Supporting punctuation without an inserted space would make it possible to follow that convention using Renovate's non-deprecated commit-message configuration options.
Current behavior
The default
commitMessagetemplate places spaces between all message components, including beforecommitMessageSuffix:Consequently,
commitMessageSuffixcannot be used for punctuation without producing an extra space.Expected behavior
There should be a supported way to append a suffix directly to the preceding component, so that:
{ "commitMessageSuffix": "." }can produce:
Current workaround
The complete
commitMessagetemplate can be overridden:{ "commitMessage": "{{{commitMessagePrefix}}} {{{commitMessageAction}}} {{{commitMessageTopic}}} {{{commitMessageExtra}}}.", "commitMessageAction": "Updated", "commitMessageLowerCase": "never" }However,
commitMessageis deprecated, and the documentation recommends customizing its individual components instead.Possible solutions
One possible solution would be a configuration option controlling the separator before
commitMessageSuffix. Alternatively, Renovate could support a punctuation-specific suffix or a template mechanism that permits suppressing the preceding space.This may also warrant clarification in the documentation if the current spacing is intentional and will not be changed.
All reactions