New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 721763 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

Owner of a directory is unable to LGTM to remove that directory

Project Member Reported by danakj@chromium.org, May 12 2017

Issue description

Presumably this breaks because the bot applies the patch which deletes the OWNERS file before checking owners approvals?

https://codereview.chromium.org/2873243002/ has jbauman approve deleting components/display_compositor, and is a member of components/display_compositor/OWNERS.

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    components/BUILD.gn
    components/display_compositor/BUILD.gn
    components/display_compositor/DEPS
    components/display_compositor/OWNERS
    components/display_compositor/buffer_queue.cc
    components/display_compositor/buffer_queue.h
    components/display_compositor/buffer_queue_unittest.cc
etc..
 
Components: -Infra>CQ Infra>Client>Chrome
Labels: -Restrict-View-Google
Possibly related:  issue 397984  .
Status: WontFix (was: Untriaged)
WontFix-ing because the related bug got WontFixed. If it's still an issue please re-open.

Sign in to add a comment