New issue
Advanced search Search tips

Issue 872381 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Do not run any analyzers on DEPS rolls (or changes not written by humans)

Project Member Reported by qyears...@chromium.org, Aug 8

Issue description

Example CL:

https://chromium-review.googlesource.com/c/chromium/src/+/1167469

The purpose of robot comments in code review is for human CL authors to make changes, but in some cases (e.g. rolls) it's probably not helpful to run analyzers at all.

We could potentially try to determine which CLs are rolls when polling new Gerrit changes, or in the analyze request.
 
Cc: diegomtzg@google.com davidben@chromium.org qyears...@chromium.org
 Issue 872355  has been merged into this issue.
Cc: -diegomtzg@google.com
Very closely related to  bug 862770 .

Sign in to add a comment