[Approved] RFR: Initial version of issue updater

Erik Helin ehelin at openjdk.java.net
Wed Nov 27 10:22:09 UTC 2019


On Wed, 27 Nov 2019 09:20:07 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:

> Hi all,
> 
> Please review this change that adds the first version of an issue updater (similar to what hgupdater currently provides for mercurial repositories).
> 
> Best regards,
> Robin
> 
> ----------------
> 
> Commits:
>  - ebed0696: Initial version of issue updater
> 
> Changes: https://git.openjdk.java.net/skara/pull/273/files
>  Webrev: https://webrevs.openjdk.java.net/skara/273/webrev.00
>   Stats: 241 lines in 5 files changed: 192 ins; 45 del; 4 mod
>   Patch: https://git.openjdk.java.net/skara/pull/273.diff
>   Fetch: git fetch https://git.openjdk.java.net/skara pull/273/head:pull/273

Very nice work Robin! So nice to see all the pieces fit into place ��

I would probably have dropped the `commit` prefix from `CommitFormatters.commitToText` and `CommitFormatters.commitToTextBrief`, so that the methods instead read `CommitFormatters.toText` and `CommitFormatters.toTextBrief`. Since they are `static` it is clear at the call sites that a commit is being formatted.

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

Approved by ehelin (Reviewer).

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


More information about the skara-dev mailing list