Browse Source

chore: review rules for FBP code to cover ya_make_team
38a89b2eb72ccf52b7112dcbc98bbdb74d765cec

zaverden 8 months ago
parent
commit
1ab14cb88b
2 changed files with 39 additions and 14 deletions
  1. 17 9
      build/conf/ts/a.yaml
  2. 22 5
      build/plugins/lib/nots/a.yaml

+ 17 - 9
build/conf/ts/a.yaml

@@ -4,18 +4,26 @@ title: Frontend build platform build/conf/ts
 arcanum:
   review:
     auto_assign: true
-    # Хотим из двух шипов от разных команд только один
-    min_approvers_count: 1
     groups:
       - name: fbp_team
         roles: frontend_build_platform:developer
       - name: ya_make_team
         roles: ya_make:developer
     rules:
-      - reviewers:
-        - name: fbp_team
-          assign: 3
-          ship: 1
-        - name: ya_make_team
-          assign: 1
-          ship: 1
+      # Если изменения от команды ya_make, то ревью на них
+      # FBP добавляется, чтобы они были в курсе, как владельцы кода
+      - authors: ya_make:developer
+        reviewers:
+          - name: ya_make_team
+            assign: 1
+            ship: 1
+            ignore_self_ship: false
+          - name: fbp_team
+            assign: 5
+            ship: 0
+      # Если изменения от кого-то стороннего, то ревью на FBP
+      - authors: "!ya_make:developer"
+        reviewers:
+          name: fbp_team
+          assign: 5
+          ship: 1

+ 22 - 5
build/plugins/lib/nots/a.yaml

@@ -4,9 +4,26 @@ title: Frontend build platform build/plugins/lib/nots
 arcanum:
   review:
     auto_assign: true
+    groups:
+      - name: fbp_team
+        roles: frontend_build_platform:developer
+      - name: ya_make_team
+        roles: ya_make:developer
     rules:
-      - reviewers:
-          name: developer
-          # мы хотим автоматом назначать всю команду
-          assign: 3
-          ship: 1
+      # Если изменения от команды ya_make, то ревью на них
+      # FBP добавляется, чтобы они были в курсе, как владельцы кода
+      - authors: ya_make:developer
+        reviewers:
+          - name: ya_make_team
+            assign: 1
+            ship: 1
+            ignore_self_ship: false
+          - name: fbp_team
+            assign: 5
+            ship: 0
+      # Если изменения от кого-то стороннего, то ревью на FBP
+      - authors: "!ya_make:developer"
+        reviewers:
+          name: fbp_team
+          assign: 5
+          ship: 1