Add os-ios, os-android etc labels to area section in resourceManageme…

…nt.yml (#106082) They only got triggered when an issue/PR was _opened_ with that label already applied, but not when it was added later on. Make it consistent with the way the area labels work.
dotnet · Aug 9, 2024 · 070fbe4 · 070fbe4
1 parent 4bd1492
commit 070fbe4
Showing 1 changed file with 92 additions and 134 deletions.
diff --git a/.github/policies/resourceManagement.yml b/.github/policies/resourceManagement.yml
@@ -321,6 +321,20 @@ configuration:
             label: area-Tools-ILLink
         - labelAdded:
             label: area-vm-coreclr
+        - labelAdded:
+            label: linkable-framework
+        - labelAdded:
+            label: size-reduction
+        - labelAdded:
+            label: arch-wasm
+        - labelAdded:
+            label: os-ios
+        - labelAdded:
+            label: os-tvos
+        - labelAdded:
+            label: os-maccatalyst
+        - labelAdded:
+            label: os-android
       then:
       - if:
         - hasLabel:
@@ -1595,54 +1609,6 @@ configuration:
 
               See info in [area-owners.md](https://github.com/dotnet/runtime/blob/main/docs/area-owners.md) if you want to be subscribed.
             assignMentionees: False
-      description: Area-owners
-    - if:
-      - payloadType: Issues
-      - labelAdded:
-          label: breaking-change
-      then:
-      - addLabel:
-          label: needs-breaking-change-doc-created
-      - addReply:
-          reply: >-
-            Added `needs-breaking-change-doc-created` label because this issue has the `breaking-change` label.
-
-
-            1. [ ] Create and link to this issue a matching issue in the dotnet/docs repo using the [breaking change documentation template](https://aka.ms/dotnet/docs/new-breaking-change-issue), then remove this `needs-breaking-change-doc-created` label.
-
-
-            Tagging @dotnet/compat for awareness of the breaking change.
-      description: Add breaking change doc label to issue
-    - if:
-      - payloadType: Pull_Request
-      - labelAdded:
-          label: breaking-change
-      - isPullRequest
-      then:
-      - addLabel:
-          label: needs-breaking-change-doc-created
-      - addReply:
-          reply: >-
-            Added `needs-breaking-change-doc-created` label because this PR has the `breaking-change` label.
-
-
-            When you commit this breaking change:
-
-
-            1. [ ] Create and link to this PR and the issue a matching issue in the dotnet/docs repo using the [breaking change documentation template](https://aka.ms/dotnet/docs/new-breaking-change-issue), then remove this `needs-breaking-change-doc-created` label.
-
-            2. [ ] Ask a committer to mail the `.NET Breaking Change Notification` DL.
-
-
-            Tagging @dotnet/compat for awareness of the breaking change.
-      description: Add breaking change doc label to PR
-    - if:
-      - or:
-        - payloadType: Issues
-        - payloadType: Pull_Request
-      - isAction:
-          action: Opened
-      then:
       - if:
         - hasLabel:
             label: linkable-framework
@@ -1658,16 +1624,8 @@ configuration:
             replyTemplate: >-
               Tagging subscribers to 'linkable-framework': ${mentionees}
 
-              See info in area-owners.md if you want to be subscribed.
+              See info in [area-owners.md](https://github.com/dotnet/runtime/blob/main/docs/area-owners.md) if you want to be subscribed.
             assignMentionees: False
-      description: '@Mention for linkable-framework'
-    - if:
-      - or:
-        - payloadType: Issues
-        - payloadType: Pull_Request
-      - isAction:
-          action: Opened
-      then:
       - if:
         - hasLabel:
             label: size-reduction
@@ -1680,16 +1638,8 @@ configuration:
             replyTemplate: >-
               Tagging subscribers to 'size-reduction': ${mentionees}
 
-              See info in area-owners.md if you want to be subscribed.
+              See info in [area-owners.md](https://github.com/dotnet/runtime/blob/main/docs/area-owners.md) if you want to be subscribed.
             assignMentionees: False
-      description: '@Mention for size-reduction'
-    - if:
-      - or:
-        - payloadType: Issues
-        - payloadType: Pull_Request
-      - isAction:
-          action: Opened
-      then:
       - if:
         - hasLabel:
             label: arch-wasm
@@ -1700,16 +1650,8 @@ configuration:
             replyTemplate: >-
               Tagging subscribers to 'arch-wasm': ${mentionees}
 
-              See info in area-owners.md if you want to be subscribed.
+              See info in [area-owners.md](https://github.com/dotnet/runtime/blob/main/docs/area-owners.md) if you want to be subscribed.
             assignMentionees: False
-      description: '@Mention for wasm'
-    - if:
-      - or:
-        - payloadType: Issues
-        - payloadType: Pull_Request
-      - isAction:
-          action: Opened
-      then:
       - if:
         - hasLabel:
             label: os-ios
@@ -1724,16 +1666,8 @@ configuration:
             replyTemplate: >-
               Tagging subscribers to 'os-ios': ${mentionees}
 
-              See info in area-owners.md if you want to be subscribed.
+              See info in [area-owners.md](https://github.com/dotnet/runtime/blob/main/docs/area-owners.md) if you want to be subscribed.
             assignMentionees: False
-      description: '@Mention for ios'
-    - if:
-      - or:
-        - payloadType: Issues
-        - payloadType: Pull_Request
-      - isAction:
-          action: Opened
-      then:
       - if:
         - hasLabel:
             label: os-android
@@ -1747,9 +1681,81 @@ configuration:
             replyTemplate: >-
               Tagging subscribers to 'arch-android': ${mentionees}
 
-              See info in area-owners.md if you want to be subscribed.
+              See info in [area-owners.md](https://github.com/dotnet/runtime/blob/main/docs/area-owners.md) if you want to be subscribed.
+            assignMentionees: False
+      - if:
+        - hasLabel:
+            label: os-tvos
+        then:
+        - mentionUsers:
+            mentionees:
+            - vitek-karas
+            - kotlarmilos
+            - ivanpovazan
+            - steveisok
+            - akoeplinger
+            replyTemplate: >-
+              Tagging subscribers to 'os-tvos': ${mentionees}
+
+              See info in [area-owners.md](https://github.com/dotnet/runtime/blob/main/docs/area-owners.md) if you want to be subscribed.
             assignMentionees: False
-      description: '@Mention for android'
+      - if:
+        - hasLabel:
+            label: os-maccatalyst
+        then:
+        - mentionUsers:
+            mentionees:
+            - vitek-karas
+            - kotlarmilos
+            - ivanpovazan
+            - steveisok
+            - akoeplinger
+            replyTemplate: >-
+              Tagging subscribers to 'os-maccatalyst': ${mentionees}
+
+              See info in [area-owners.md](https://github.com/dotnet/runtime/blob/main/docs/area-owners.md) if you want to be subscribed.
+            assignMentionees: False
+      description: Area-owners
+    - if:
+      - payloadType: Issues
+      - labelAdded:
+          label: breaking-change
+      then:
+      - addLabel:
+          label: needs-breaking-change-doc-created
+      - addReply:
+          reply: >-
+            Added `needs-breaking-change-doc-created` label because this issue has the `breaking-change` label.
+
+
+            1. [ ] Create and link to this issue a matching issue in the dotnet/docs repo using the [breaking change documentation template](https://aka.ms/dotnet/docs/new-breaking-change-issue), then remove this `needs-breaking-change-doc-created` label.
+
+
+            Tagging @dotnet/compat for awareness of the breaking change.
+      description: Add breaking change doc label to issue
+    - if:
+      - payloadType: Pull_Request
+      - labelAdded:
+          label: breaking-change
+      - isPullRequest
+      then:
+      - addLabel:
+          label: needs-breaking-change-doc-created
+      - addReply:
+          reply: >-
+            Added `needs-breaking-change-doc-created` label because this PR has the `breaking-change` label.
+
+
+            When you commit this breaking change:
+
+
+            1. [ ] Create and link to this PR and the issue a matching issue in the dotnet/docs repo using the [breaking change documentation template](https://aka.ms/dotnet/docs/new-breaking-change-issue), then remove this `needs-breaking-change-doc-created` label.
+
+            2. [ ] Ask a committer to mail the `.NET Breaking Change Notification` DL.
+
+
+            Tagging @dotnet/compat for awareness of the breaking change.
+      description: Add breaking change doc label to PR
     - if:
       - payloadType: Pull_Request
       - or:
@@ -1798,54 +1804,6 @@ configuration:
       - addLabel:
           label: no-recent-activity
       description: Manual Issue Cleanup
-    - if:
-      - or:
-        - payloadType: Issues
-        - payloadType: Pull_Request
-      - isAction:
-          action: Opened
-      then:
-      - if:
-        - hasLabel:
-            label: os-tvos
-        then:
-        - mentionUsers:
-            mentionees:
-            - vitek-karas
-            - kotlarmilos
-            - ivanpovazan
-            - steveisok
-            - akoeplinger
-            replyTemplate: >-
-              Tagging subscribers to 'os-tvos': ${mentionees}
-
-              See info in area-owners.md if you want to be subscribed.
-            assignMentionees: False
-      description: '@Mention for tvos'
-    - if:
-      - or:
-        - payloadType: Issues
-        - payloadType: Pull_Request
-      - isAction:
-          action: Opened
-      then:
-      - if:
-        - hasLabel:
-            label: os-maccatalyst
-        then:
-        - mentionUsers:
-            mentionees:
-            - vitek-karas
-            - kotlarmilos
-            - ivanpovazan
-            - steveisok
-            - akoeplinger
-            replyTemplate: >-
-              Tagging subscribers to 'os-maccatalyst': ${mentionees}
-
-              See info in area-owners.md if you want to be subscribed.
-            assignMentionees: False
-      description: '@Mention for maccatalyst'
     - if:
       - payloadType: Issues
       - or: