search

OSS-Docs-Tools / code-owner-self-merge

A GitHub Action for letting CODEOWNERS merge PRs via green PR reviews
MIT License
53 stars 16 forks source link

Convert `lgtm` to a command-like word #34

Open saschanaz opened 2 years ago

saschanaz commented 2 years ago

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

orta commented 2 years ago

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