Internal Rotation Script
PiperOrigin-RevId: 363276312
Change-Id: I330bac5661689b3c3b249c479961473a18dafe61
diff --git a/tensorflow/tools/api/golden/api_owners_rotation.gwsq b/tensorflow/tools/api/golden/api_owners_rotation.gwsq
index 87bbc17..4a5f8c8 100644
--- a/tensorflow/tools/api/golden/api_owners_rotation.gwsq
+++ b/tensorflow/tools/api/golden/api_owners_rotation.gwsq
@@ -1,7 +1,15 @@
-send_cls_to('tf-api-owners+reviews');
+send_cls_to('tf-api-owners+reviews', move_to_cc=False);
+send_cls_to('tf-api-owners', move_to_cc=False);
+// The full TF API Owners review process is described at go/tf-api-review-process
define Main {
+ // The TF API Owners only review CLs with at least one LGTM
+ if (!has_lgtm()) {
+ add_comment('Please get at least one LGTM first before asking for a review from the TF API Owners (go/tf-api-review-process). When ready, add tf-api-owners+reviews as a reviewer.');
+ return;
+ }
+
add_comment("The API owners use a rotation based system (go/tf-api-review-process). Assigning several owners to follow along and review.\nFor the owners, if you are the second owner LGTM, please approve the CL.");
with ("tensorflow\.tpu") {
@@ -23,4 +31,6 @@
reassign_to_list(choose_n(2, google_group('tf-api-owners-rotation')), minimize_reviewer_count=True);
set_tag('WANT_LGTM', 'all');
+
+ add_cc(google_group('tf-api-owners+reviews'));
}
