2018-01-25 22:55 GMT+01:00 Mariatta Wijaya <mariatta.wij...@gmail.com>: > My problem has been that I almost always still need to rewrite the commit > message. > Especially when someone wrote "fix a typo" or "fix several typos".
That's the main drawback of GitHub compared to Gerrit. On Gerrit, the commit message is at the same level than other changes. We can comment the commit message as we comment the change. Note: On Gerrit, it's even possible to edit the commit message online (using the web browser). > If it automatically merges, then there's no opportunity to adjust the commit > message. Maybe we need a tool to preview the future commit message, and let the developer modify it. For example, a merge request would add a comment to the PR with the future commit message and then asks to validate the commit message, or let the developer to override (modify) it. Example: --- Comment 4, dev: "Merge" Comment 5, bot: "bpo-123: Cool fix" Comment 6, dev: "Confirm merge" Comment 7, GitHub: "PR merged, commit xxx" --- > So I suggest the option to provide the proper commit message to the > mergebot. Sometimes, the commit message is just fine and can be used unchanged ;-) > If not provided, I guess we'll use the GitHub PR title and description. Sometimes, I see a typo in the commit message after I push a change. In this case, I quickly amend my commit and push my commit again. But the PR description keeps my original commit message with the typo. So the PR description is not reliable. It doesn't seem to be used by GitHub to generate the commit message, GitHub combines commit messages of the PR commits. Victor _______________________________________________ Python-Dev mailing list Python-Dev@python.org https://mail.python.org/mailman/listinfo/python-dev Unsubscribe: https://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
