RFR: git-webrev: show commit messages

Robin Westberg rwestberg at openjdk.java.net
Mon Feb 17 12:14:15 UTC 2020


On Mon, 17 Feb 2020 10:48:48 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hi all,
> 
> please review this patch that adds commit messages to webrevs (if applicable).
> The current `MetadataFormatter` is a bit bare-bones, I plan to enhance it in
> future patches. I had to implement a few overloads of `commitMetadata` to
> support this feature, hence the changes to `vcs`.
> 
> Testing:
> - Added unit tests for `commitMetadata`
> - Manual testing of `git-webrev` for both Git and Mercurial repositories
> 
> Thanks,
> Erik

Looks good!

-------------

Marked as reviewed by rwestberg (Reviewer).

PR: https://git.openjdk.java.net/skara/pull/441


More information about the skara-dev mailing list