I stumbled over commits that carry the author's Reviewed-by.
There may be cases where the recorded author isn't the lone author, and
the recorded author did some meaningful review of the patch's parts that
are not theirs. Mind that we do need all authors to provide their
Signed-off-by.
When the only Signed-off-by is from the recorded author, and there's
also their Reviewed-by, the Reviewed-by is almost certainly bogus.
Now, accidents happen, no big deal, etc., etc. I post this to hopefully
help reduce the accident rate :)
Here's my quick & sloppy search for potentially problematic uses of
Reviewed-by:
$ git-log --since 'two years ago' | awk -F: '/^commit / { commit=$0 } /^Author:
/ { guy=$2 } /^ Reviewed-by: / { if ($2 == guy) { print commit; print guy }
}'
