Hacker News Reader

Isn't that supposed to be the reviewer's name, rather than the person who made the change?

Not as originally implemented for use in the Linux kernel. It's meant to identify the original author, who is signing off on agreeing to contribute the patch as open source.

See https://www.kernel.org/doc/Documentation/SubmittingPatches section 11: “Sign your work”.

Ah. I've seen it show up in the GUIs for a coworker or two, but I've never actually seen it filled in.

Reply on news.ycombinator.com