Integrated: Remove mention of /integrate from the start of the pre-integration message.
Jorn Vernee
jvernee at openjdk.java.net
Mon Sep 7 13:06:06 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
This pull request has now been integrated.
Changeset: 55c4ce77
Author: Jorn Vernee <jvernee at openjdk.org>
URL: https://git.openjdk.java.net/skara/commit/55c4ce77
Stats: 3 lines in 1 file changed: 0 ins; 0 del; 3 mod
Remove mention of /integrate from the start of the pre-integration message.
Reviewed-by: rwestberg, ehelin
-------------
PR: https://git.openjdk.java.net/skara/pull/801
More information about the skara-dev
mailing list