Closed mbarbero closed 3 months ago
Sounds reasonable. I was already thinking of applying something similar as dependabot.
Using @otterdog is not a good idea, as it will tag the user otterdog, which exists:
Some other prefixes that might work:
!
?
.
/
#
should not be used imho as it is used to reference Tickets / PRsmy preference would be .
, so the commands would look like .otterdog validate
, or just keep using /
with scope, e.g. /otterdog validate
Good catch about @otterdog
. I tend to prefer the /
prefix (with scope, ie /otterdog
) as it's closer to other "command language", like in IRC, Slack, or Discord.
👍
Currently, the commands for Otterdog, such as verify, merge, etc., are not scoped. They can be placed anywhere within the comment body as long as they start with a
/
. This approach could potentially lead to confusion, unintended consequences, or even conflicts with other applications.This ticket aims to discuss the adoption of scoped commands, similar to how Dependabot operates (https://docs.github.com/en/code-security/dependabot/working-with-dependabot/managing-pull-requests-for-dependency-updates#managing-dependabot-pull-requests-with-comment-commands), for example:
Thoughts?