[Webkit-unassigned] [Bug 242841] GitHub shows reviewer name more prominently than patch author

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=242841

Cameron McCormack (:heycam) <heycam at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |heycam at apple.com

--- Comment #1 from Cameron McCormack (:heycam) <heycam at apple.com> ---
The Github UI shows certain commit metadata specially, like the committer and authors, just below the commit message, but I don't know if anything else (including any structured git commit message trailer, like a "Signed-Off-By:" line) gets shown specially.

Without asking Github to make changes to their UI, the only options I can think of would be:

1. Move the "Reviewed by" line to the end of the commit message, so that it's closer to the author's username in the Github UI.
2. Add the author details into the commit message too.

The reviewer information is pretty important, and I like where it is in the commit message (towards the top).  Adding the author name to the commit message duplicates information from the git commit metadata, and duplicates the information shown when looking at the commit in other tools (like `git log`).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220717/74c2a22c/attachment.htm>