code-owner-self-merge icon indicating copy to clipboard operation
code-owner-self-merge copied to clipboard

Convert `lgtm` to a command-like word

Open saschanaz opened this issue 3 years ago • 2 comments

Because a comment like "lgtm but..." accidentally causes a merge. 😬

saschanaz avatar Feb 19 '22 15:02 saschanaz

I'd skill kinda prefer it feeling natural, that said I added protection against 'lgtm but' on HEAD - https://github.com/OSS-Docs-Tools/code-owner-self-merge/commit/8714b4990278162db0fb87717dd4a1e56fbd7a36

orta avatar Feb 28 '22 08:02 orta