New issue
Advanced search Search tips

Issue 597268 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner: ----
Closed: Mar 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

CQ Dry Run - No LGTM for DEPS changes should be ignored.

Project Member Reported by robliao@chromium.org, Mar 23 2016

Issue description

For changes that change DEPS, you'll see this in the error log:

** Presubmit Messages **
This is a CQ dry run, skipping OWNERS check

** Presubmit ERRORS **
Missing LGTM from OWNERS of dependencies added to DEPS:
    [Dependencies]

These should also be ignored as part of the CQ Dry Run
 
Summary: CQ Dry Run - No LGTM for DEPS changes should be ignored. (was: CQ Dry Run - No LGTM for DEPS changes should be ignore.)

Comment 2 Deleted

Labels: -Build-CommitQueue Infra-CommitQueue
Changing the label to the correct one, so it shows up on our daily triage.
Mergedinto: 594127
Status: Duplicate (was: Untriaged)

Comment 5 by aga...@chromium.org, Apr 26 2016

Components: Infra>CQ
Labels: -Infra-CommitQueue

Sign in to add a comment