commit c2756dca203815cd5c2d16270847432c97a670c8
author Tim van der Lippe <tvanderlippe@chromium.org> Wed Dec 08 12:17:32 2021 +0000
committer Devtools-frontend LUCI CQ <devtools-frontend-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Dec 08 13:05:28 2021 +0000
tree c55f5471bee26b2124f3e6c75cf15e6f6f08bb41
parent e8229b2f2b19bb0e4d50b1d2e3c4632ba65f3138

Consider all OWNERS changes as exclusive

I have verified locally that the check will error if you create a commit
that includes both a nested OWNERS file and any other change.

R=jacktfranklin@chromium.org

Fixed: 1029775
Change-Id: Ied7875aeb5b805594dc89107ff5e5a1827bf62e8
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/3322832
Commit-Queue: Tim Van der Lippe <tvanderlippe@chromium.org>
Auto-Submit: Tim Van der Lippe <tvanderlippe@chromium.org>
Reviewed-by: Jack Franklin <jacktfranklin@chromium.org>
Commit-Queue: Jack Franklin <jacktfranklin@chromium.org>

PRESUBMIT.py

diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index 72b55cc..f3a1982 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -75,6 +75,8 @@
         return False
 
     def FileIsInDir(file, dirs):
+        if file.endswith('OWNERS') and 'OWNERS' in dirs:
+            return True
         for dir in dirs:
             if IsParentDir(file, dir):
                 return True
@@ -89,7 +91,7 @@
             'package-lock.json',
             input_api.os_path.join('scripts', 'deps', 'manage_node_deps.py'),
         ],
-        ['OWNERS', input_api.os_path.join('config', 'owner')],
+        ['OWNERS'],
     ]
 
     affected_files = input_api.LocalPaths()