RFR: 1828: RFR emails still be sent when PR is in draft state

Erik Joelsson erikj at openjdk.org
Wed Mar 15 14:18:24 UTC 2023


On Wed, 15 Mar 2023 13:50:16 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> Hi all,
> 
> This patch blocks the RFR email when new commits are pushed to a draft PR. And the test case is added. too.
> 
> But it doesn't solve the synchronization problem that Erik mentioned in [PR-1469](https://github.com/openjdk/skara/pull/1469#issuecomment-1425916242). Because the bot only generates one email one time regradless of the number of the commits.
> 
> Thanks for the review.
> 
> Best Regards,
> -- Guoxiong

I think this is fine. Perfect synchronization on commit emails isn't as important as for comments.

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

Marked as reviewed by erikj (Lead).

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


More information about the skara-dev mailing list