<div dir="ltr">I just merged the PR and went with "I have made the requested changes; please review again". Figured this makes people aware that they are to have addressed the changes before requesting a review and has them saying "please". :) Plus there's no way anyone will accidentally type that in conversation on a pull request.<br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 10 Oct 2017 at 00:09 Ethan Furman <<a href="mailto:ethan@stoneleaf.us">ethan@stoneleaf.us</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 10/08/2017 09:44 AM, Brett Cannon wrote:<br>
<br>
> I actually wouldn't want the bot name in the trigger phrase since you're not addressing the bot but the reviewer(s). So<br>
> using something that is unambiguous as a trigger phrase like "please re-review" or "please review again" that won't come<br>
> up in conversation about what is required should be enough to be unambiguous of the intent of the commenter as well has<br>
> not seeming quite so forced.<br>
<br>
You're addressing the bot to notify the reviewers.  It's like asking one's secretary to schedule an appointment with<br>
one's peers.<br>
<br>
--<br>
~Ethan~<br>
_______________________________________________<br>
core-workflow mailing list<br>
<a href="mailto:core-workflow@python.org" target="_blank">core-workflow@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/core-workflow" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/core-workflow</a><br>
This list is governed by the PSF Code of Conduct: <a href="https://www.python.org/psf/codeofconduct" rel="noreferrer" target="_blank">https://www.python.org/psf/codeofconduct</a><br>
</blockquote></div>