RFR: Remove mention of /integrate from the start of the pre-integration message.
Robin Westberg
rwestberg at openjdk.java.net
Mon Sep 7 12:43:16 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 for fixing!
-------------
Marked as reviewed by rwestberg (Reviewer).
PR: https://git.openjdk.java.net/skara/pull/801
More information about the skara-dev
mailing list