Make Reviewed-by trailer token a configurable option #10872

New Issue

GiteaMirror · 2025-11-02T09:20:48-06:00

GiteaMirror commented

2025-11-02 09:20:48 -06:00

Originally created by @corytodd on GitHub (May 18, 2023).

Feature Description

When using the default merge message, the reviewer annotation uses Review-by as the git trailer token. Our workflow would like to use the Acked-by token instead. We propose making this a global configuration in the ini unless someone else sees value in this feature and would like it to be per-repo.

Example:

The current default merge message on pull requests looks something like this:

Merge pull request '$title' (!$index) from $head into $source
 
Reviewed-on: [#index]($url)
Reviewed-by: Some User <some.user@example.com>

We would like to replace the hard-coded Reviewed-by string with a value that administrators can define in the ini file. We would keep the default value as Reviewed-by to make this opt-in.

Screenshots

No response

Originally created by @corytodd on GitHub (May 18, 2023). ### Feature Description When using the default merge message, the reviewer annotation uses Review-by as the git trailer token. Our workflow would like to use the Acked-by token instead. We propose making this a global configuration in the ini unless someone else sees value in this feature and would like it to be per-repo. Example: The current default merge message on pull requests looks something like this: ``` Merge pull request '$title' (!$index) from $head into $source Reviewed-on: [#index]($url) Reviewed-by: Some User <some.user@example.com> ``` We would like to replace the hard-coded Reviewed-by string with a value that administrators can define in the ini file. We would keep the default value as `Reviewed-by` to make this opt-in. ### Screenshots _No response_

GiteaMirror added the type/proposal type/feature labels 2025-11-02 09:20:48 -06:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/gitea#10872