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