From 5167030a3a0f4024ed6bc95a340a3da7736b6c65 Mon Sep 17 00:00:00 2001 From: Joey Perrott Date: Tue, 30 Mar 2021 11:38:09 -0700 Subject: [PATCH] ci: remove *.bazel from dev-infra ownership (#41382) Historically all .bazel files were owned by dev-infra to assist in ensuring that consistency was maintained in how rules were being used. As this now has reached a point of maturity in the repository where the check is mostly a rubber stamping, this requirement can be removed. Notably this is different than the requirement of dev-infra approval for .bzl files as these contain the implementation of macros rather than just the usage. PR Close #41382 --- .pullapprove.yml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/.pullapprove.yml b/.pullapprove.yml index fc6cfd8e3d..eb094b566b 100644 --- a/.pullapprove.yml +++ b/.pullapprove.yml @@ -1166,7 +1166,7 @@ groups: conditions: - *can-be-global-approved - > - contains_any_globs(files.exclude("packages/compiler-cli/**/BUILD.bazel"), [ + contains_any_globs(files, [ '*', '.circleci/**', '.devcontainer/**', @@ -1203,8 +1203,7 @@ groups: 'tools/utils/**', 'tools/yarn/**', 'tools/*', - '**/*.bzl', - '**/*.bazel' + '**/*.bzl' ]) reviewers: users: