RFR: 1789: Skara PR commands are not interpreted on Github review comments [v2]

David Holmes dholmes at openjdk.org
Thu Jan 19 06:44:34 UTC 2023


On Wed, 18 Jan 2023 21:41:03 GMT, Zhao Song <zsong at openjdk.org> wrote:

>> A user reported that Skara bots were not responding to commands in review comments or the body of a review. 
>> 
>> This patch addresses that issue by ensuring that the Skara bots can correctly detect and interpret commands.
>
> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
> 
>   sort comments by created time

I also have misgivings about scanning review comments for PR commands - so it seems there is consensus there. 

Even in the main review comment what gets scanned - all the text? Only the start of lines? I would worry about text being misconstrued as a command. This would especially be true for skara PRs where review comments may well refer to PR commands as part of the discussion.

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

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


More information about the skara-dev mailing list