RFR: Remove mention of /integrate from the start of the pre-integration message.

Erik Helin ehelin at openjdk.java.net
Mon Sep 7 13:02:45 UTC 2020


On Mon, 7 Sep 2020 10:15:51 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

> Hi,
> 
> In response to some of the discussion on: https://github.com/openjdk/jdk/pull/26 I suggest changing the pre-integration
> comment to remove the mention of `/integrate` at the start, where it reads:
>     type `/integrate` in a new comment to proceed
> 
> Since `/integrate` is also mentioned at the end of the same message (with a clearer description of what it does
> [1](https://github.com/openjdk/skara/blob/master/bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java#L603-L610)),
> this is an attempt to encourage users to read till the end of the message before typing `/integrate`.  Thanks,
> Jorn

Looks good, thanks Jorn!

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

Marked as reviewed by ehelin (Reviewer).

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


More information about the skara-dev mailing list