Remove special handling of "LGTM" in review comments
Don't automatically select the highest available Code-Review score
when the reviewer types "LGTM" in the review comment.
- The text when the reviewer selects +1 is "Looks good to me, but
someone else must approve". In other words, it's false to assume
that "LGTM" means +2 when it could also mean +1.
- The UI shouldn't override input that the reviewer has already made
(i.e. explicitly selecting the Code-Review +1 score)
- For Gerrit installations tied to a CI system that has an "auto
submit on +2" feature, this could potentially trigger a submit when
unintended.
Bug: Issue 3130
Change-Id: I73692dcf3f82b29a17cb9ada1fe494cc319587c8
2 files changed
