On 8 October 2017 at 07:38, Donald Stufft <don...@stufft.io> wrote:

> 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”.
>
> 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.
>

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).

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).

Cheers,
Nick.

-- 
Nick Coghlan   |   ncogh...@gmail.com   |   Brisbane, Australia
_______________________________________________
core-workflow mailing list
core-workflow@python.org
https://mail.python.org/mailman/listinfo/core-workflow
This list is governed by the PSF Code of Conduct: 
https://www.python.org/psf/codeofconduct

Reply via email to