<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 8 October 2017 at 07:38, Donald Stufft <span dir="ltr"><<a href="mailto:donald@stufft.io" target="_blank">donald@stufft.io</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Currently the workflow for CPython development requires people to say 'I didn't expect the Spanish Inquisition’ in order to request a re-review of their work. Can we please use a phrase for this that makes more sense rather than, as Alex put it, “magic inside baseball language”.<br>
<br>
In jokes can be fun when they’re able to essentially be just noise to people who aren’t part of the in crowd (e.g. the bot name being one is fine) but they’re kind of crummy when a core part of the developer experience or API. Leave it in as an Easter egg if you like (and probably should for backwards compatibility anyways), but please make something else be the primary phrase.<br></blockquote><div><br></div>I'd agree with this (especially since references to the Spanish inquisition aren't going to be funny for folks that are still facing religious persecution).</div><div class="gmail_quote"><br></div><div class="gmail_quote">Having the bot name in the trigger phrase is a good way to avoid accidental activation, so something like "Bedevere: ready for review" would be good (and, as Donald notes, it's fine to keep the current phrase as a secondary trigger - it just shouldn't be the main documented one).</div><div class="gmail_quote"><br></div><div class="gmail_quote">Cheers,</div><div class="gmail_quote">Nick.<br></div><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Nick Coghlan   |   <a href="mailto:ncoghlan@gmail.com" target="_blank">ncoghlan@gmail.com</a>   |   Brisbane, Australia</div>
</div></div>