RFR: SKARA-1119: A bot should remind a PR author not to force push [v2]
Guoxiong Li
gli at openjdk.java.net
Wed Jun 1 08:30:22 UTC 2022
On Tue, 31 May 2022 18:16:13 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
>> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix suggestion comment.
>
> bots/notify/src/main/java/org/openjdk/skara/bots/notify/prbranch/PullRequestBranchNotifier.java line 39:
>
>> 37: Please don't rebase and force-push to your branch of this PR because it invalidates previous review comments. \
>> 38: To keep track of your changes incrementally, you only need to merge the target branch (optionally), \
>> 39: commit your new change and push normally. The bot can squash them as a single commit when integrating.
>
> Suggestion:
>
> Please do not rebase or force-push to an active PR as it invalidates existing review comments. \
> All changes will be squashed into a single commit automatically when integrating. \
> See [OpenJDK Developers’ Guide](https://openjdk.java.net/guide/#working-with-pull-requests).
Fixed
-------------
PR: https://git.openjdk.java.net/skara/pull/1328
More information about the skara-dev
mailing list