Skip to content

Commit 697c982

Browse files
committed
respond to feedback
1 parent b50a7c3 commit 697c982

File tree

1 file changed

+13
-28
lines changed

1 file changed

+13
-28
lines changed

.github/policies/disallow-edits.yml

Lines changed: 13 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ configuration:
124124

125125
- description: Alert authors to edits in the /articles/reliability folder.
126126
if:
127-
# If a PR in the articles/reliability folder was opened or had the ready-to-merge label added, and the PR author isn't Anastasia...
127+
# If a PR in the articles/reliability folder was opened or had the ready-to-merge label added, and the PR author isn't Anastasia or John...
128128
- payloadType: Pull_Request
129129
- or:
130130
- isAction:
@@ -135,39 +135,24 @@ configuration:
135135
matchAny: true
136136
pattern: articles/reliability/*
137137
- not:
138-
isActivitySender:
139-
user: annaharris-ms
138+
or:
139+
- isActivitySender:
140+
user: anaharris-ms
141+
- isActivitySender:
142+
user: johndowns
140143
then:
141-
# Add the needs-human-review label, remove the ready-to-merge label (if necessary), mention all authors in this folder, and add a reply asking the PR author not to sign off on the PR.
142-
- addLabel:
143-
label: needs-human-review
144-
- removeLabel:
145-
label: ready-to-merge
144+
# Add the needs-human-review label, remove the ready-to-merge label (if necessary), mention Anastasia and John, and add a reply asking the PR author not to sign off on the PR.
146145
- mentionUsers:
147146
mentionees:
148-
- annaharris-ms
149-
- AbdullahBell
150-
- jfaurskov
151-
- maud-lv
152-
- vladai78
153-
- RoseHJM
154-
- dlepow
155-
- nzthiago
156-
- whhender
157-
- adishachar
158-
- surbhijain
159-
- jovinson-ms
160-
- roygara
161-
- craigshoemaker
162-
- mattgotteiner
163-
- paulinbar
164-
- sethmanheim
165-
- johnmic
166-
- gkasar
167-
- tomvcassidy
147+
- anaharris-ms
148+
- johndowns
168149
replyTemplate: >-
169150
Tagging authors for this folder: ${mentionees}
170151
- addReply:
171152
reply: >-
172153
@${issueAuthor} - Please do NOT sign off on this pull request. The article author will sign off for you.
154+
- addLabel:
155+
label: needs-human-review
156+
- removeLabel:
157+
label: ready-to-merge
173158

0 commit comments

Comments
 (0)