Code Review Rules

.pullapprove.yml

@@ -237,6 +237,85 @@ groups:
     reviews:
       request: 0 # Do not auto-add
 
+  ############################################################
+  #  Path specific review requirments. A PR must meet the
+  #  review requirements corresponding each path in the
+  #  in the CUSTOM_REVIEW_PATHS list that it touches. If it
+  #  touches a region not covered by any custom review paths, 
+  #  base condition is required.
+  ############################################################
+  shared-reviewers-scripts-build:
+    type: optional
+    conditions:
+      - files.include('*')
+    reviewers:
+      users:
+        - andy31415
+    reviews:
+      request: 0 # Do not auto-add
+  shared-reviewers-integrations-docker:
+    type: optional
+    conditions:
+      - files.include('*')
+    reviewers:
+      users:
+        - andy31415
+    reviews:
+      request: 0 # Do not auto-add
+
+  ############################################################
+  #  Review requirements for PRs touching examples/chef*
+  #  One Googler requirement is sufficient.
+  ############################################################
+  chef-required-reviewers:
+    type: required
+    conditions:
+      - "'examples/chef/*' in files"
+    requirements:
+      - len(groups.approved.include('shared-reviewers-google')) >= 1 or len(groups.approved.include('shared-reviewers-*')) >= 2  
+    reviews:
+      required: 0
+
+  ############################################################
+  #  Review requirements for PRs touching scripts/build*.
+  #  One shared-reviewers-scripts-build review is sufficient.
+  ############################################################
+  scripts-build-required-reviewers:
+    type: required
+    conditions:
+      - "'scripts/build/*' in files"
+    requirements:
+      - len(groups.approved.include('shared-reviewers-scripts-build')) >= 1 or len(groups.approved.include('shared-reviewers-*')) >= 2
+    reviews:
+      required: 0
+
+  ############################################################
+  #  Review requirements for PRs touching integrations/docker*
+  #  One shared-reviewers-integrations-docker review is
+  #  sufficient.
+  ############################################################
+  integrations-docker-required-reviewers:
+    type: required
+    conditions:
+      - "'integrations/docker/*' in files"
+    requirements:
+      - len(groups.approved.include('shared-reviewers-integrations-docker')) >= 1 or len(groups.approved.include('shared-reviewers-*')) >= 2
+    reviews:
+      required: 0
+
+  ############################################################
+  #  Review requirements for PRs touching integrations/cloudbuild*
+  #  One Google review is sufficient.
+  ############################################################
+  integrations-cloudbuild-required-reviewers:
+    type: required
+    conditions:
+      - "'integrations/cloudbuild/*' in files"
+    requirements:
+      - len(groups.approved.include('shared-reviewers-google')) >= 1 or len(groups.approved.include('shared-reviewers-*')) >= 2
+    reviews:
+      required: 0
+
   ############################################################
   #  Base Required Reviewers
   ############################################################
@@ -247,6 +326,14 @@ groups:
       This is the main group of required reviews for general pull
       requests.
     type: required
+    conditions:
+      # If PR touches a region not covered by any custom review paths, base condition is required.
+      - "len(files\
+              .exclude('examples/chef/*')\
+              .exclude('scripts/build/*')\
+              .exclude('integrations/docker/*')\
+              .exclude('integrations/cloudbuild/*')\
+            ) >= 1"
     requirements:
       - len(groups.approved.include('shared-reviewers-*')) >= 2
     reviews:

examples/chef/common/stubs.cpp

@@ -6,6 +6,10 @@
 #include <app/util/config.h>
 #include <lib/core/DataModelTypes.h>
 
+/**
+ * TODO: Remove this comment.
+ */
+
 using chip::app::DataModel::Nullable;
 
 using namespace chef;

integrations/cloudbuild/chef.yaml

@@ -89,3 +89,5 @@ artifacts:
 options:
     machineType: "E2_HIGHCPU_32"
     diskSizeGb: 500
+
+# TODO: Remove this comment.

integrations/docker/build.sh

@@ -121,3 +121,5 @@ if [ -f "$DOCKER_VOLUME_PATH" ]; then
 fi
 
 exit 0
+
+# TODO: Remove this comment.

scripts/build/BUILD.gn

@@ -72,3 +72,4 @@ pw_python_package("build_examples") {
     "build/test_target.py",
   ]
 }
+# TODO: Remove this comment.