Consider cleaning up commit messages by eliminating merge request templates
Let's make the default merge commit message (when merging a merge request from the UI) more closely match git conventions. This means:
- Making the commit message as close to a normal merge commit message as possible;
- BUT including the MR title;
- AND including the issues closed by the MR;
- AND including the link / reference to the MR itself.
There are two additional concerns here, which I'll put in the order they matter to me personally
- Sometimes people mention the issue that they are fixing in the merge request, but don't add the issue closing pattern. This means that merging won't actually close that issue or issues. It would be nice to get a warning in this case, and be able to easily add those issues (with closing pattern) to the commit message.
- I can almost guarantee that this will break someone's workflow, so there should be a way to use the MR description as the commit message. This doesn't have to be a setting, because we don't like to add settings, but just a feature in the UI that someone can use on a case-by-case basis.
Original description
I understand how merge request templates force a person to check off everything, but I find it really pollutes the commit logs and makes it harder to read a history via
git log
. I miss clean commit logs in a form such as:http://chris.beams.io/posts/git-commit/
What are people thoughts? Should we consider an alternative approach?
/cc: @jschatz1, @rymai, @rspeicher, @DouweM, @undefined