RFR: 1076: Not possible to use 'Backport <sha>' if original bug doesn't have public sha [v3]

Erik Joelsson erikj at openjdk.java.net
Fri Jun 11 19:59:49 UTC 2021


On Fri, 11 Jun 2021 18:53:50 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Erik Joelsson has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review comments
>
> bots/pr/src/test/java/org/openjdk/skara/bots/pr/BackportTests.java line 1458:
> 
>> 1456:             assertEquals(List.of(), message.contributors());
>> 1457:             assertEquals(List.of(), message.summaries());
>> 1458:             assertEquals(List.of(), message.additional());
> 
> It might be good to also check that the commit message does not contain "Backport of", or is this covered by checking that `message.additional()` is empty?

Good point. I just removed the assert for the existence of the original hash without thinking about inverting that assert.

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

PR: https://git.openjdk.java.net/skara/pull/1186


More information about the skara-dev mailing list