RFR: 1516: Command 'git pr integrate' gets error message 'time out'

Magnus Ihse Bursie ihse at openjdk.org
Tue Aug 9 15:34:40 UTC 2022


On Tue, 9 Aug 2022 13:05:17 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> Hi all,
> 
> I tried the command `git pr integrate 1341` to integrate SKARA-1495. The command replies `error: timed out waiting for response to /integrate command`. It is because the bot added a comment `@lgxbslgx Pushed as commit XXX.` so that this line is not started with `Pushed as commit`.
> 
> This patch fixes the bug by using a pattern matcher instead of judging the beginning of the line. And I verified this patch by using the command `git pr integrate 1344` to integrate SKARA-1463 just now. It works well and replies `@lgxbslgx Pushed as commit 35b23b882393c33565e49ec0399adba7e8eb2f5b`. But I have not verified the sponsor flow because I don't have a change to do that now.
> 
> Thanks for taking the time to review.
> 
> Best Regards,
> -- Guoxion

Is there no tests for this? Can you please add one, if not, or update it, if it does not test this properly.

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

PR: https://git.openjdk.org/skara/pull/1352


More information about the skara-dev mailing list