New issue
Advanced search Search tips

Issue 923897 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

allow for lgtm anywhere in the text, but require some escape character

Project Member Reported by jochen@chromium.org, Yesterday (37 hours ago)

Issue description

currently, gerrit will R+1 if the message starts with lgtm.

I think this is both too restrictive, and still error prone. Instead, why not accept lgtm anywhere but require an escape character in front of it, e.g. #lgtm or similar?
 

Sign in to add a comment