On Sat, Jul 1, 2017 at 5:24 PM, Dan Kohn <[email protected]> wrote: > https://github.com/coreinfrastructure/best-practices-badge is a user > of the https://github.com/probot/dco bot which checks that commits > have a signoff. The issue is that there is no `--signoff` option in > git for merge commits, which is a standard part of our workflow with > feature branches. Here is a workflow where we currently get stuck:
[...] > Or, I could manually add the Signoff line to the proposed git merge > commit message, which would allow me to skip the `--amend` step. Perhaps you could use a prepare-commit-msg hook (or maybe a commit-msg hook) to automatically add your Signoff line to any commit message if it isn't there already. In the prepare-commit-msg hook sample there is already commented out code to do that: https://github.com/git/git/blob/master/templates/hooks--prepare-commit-msg.sample#L35-L36 Alternatively you might want to use `git interpret-trailers` to do that or more fancy trailer related things. > Could you please add a `--signoff` option to `git merge`? I am not opposed to add a `--signoff` option to `git merge`, but I think the main plan to improve git in this area has been to first make it possible for git commands that can create commits to accept options like "--trailer 'Signed-off-by: Alice <[email protected]>'" and to pass them to `git interpret-trailers` (or its underlying code).
