Skip to content

Commit 45f8a13

Browse files
author
Darien Lin
committed
ci: requiring two org members to approve for PR merge; preventing in org sock puppet accounts
1 parent dfb2b7f commit 45f8a13

File tree

2 files changed

+264
-0
lines changed

2 files changed

+264
-0
lines changed

.github/workflows/members.json

Lines changed: 233 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,233 @@
1+
[
2+
{
3+
"login": "aaron-prindle"
4+
},
5+
{
6+
"login": "akulkapoor-google"
7+
},
8+
{
9+
"login": "alicejli"
10+
},
11+
{
12+
"login": "alphanota"
13+
},
14+
{
15+
"login": "bendory"
16+
},
17+
{
18+
"login": "blakeli0"
19+
},
20+
{
21+
"login": "bobcallaway"
22+
},
23+
{
24+
"login": "bobcatfish"
25+
},
26+
{
27+
"login": "briandealwis"
28+
},
29+
{
30+
"login": "burkedavison"
31+
},
32+
{
33+
"login": "Camila-B"
34+
},
35+
{
36+
"login": "chanseokoh"
37+
},
38+
{
39+
"login": "ChrisGe4"
40+
},
41+
{
42+
"login": "ChristopherFry"
43+
},
44+
{
45+
"login": "chuangw6"
46+
},
47+
{
48+
"login": "container-tools-bot"
49+
},
50+
{
51+
"login": "cvgw"
52+
},
53+
{
54+
"login": "dangazineu"
55+
},
56+
{
57+
"login": "Darien-Lin"
58+
},
59+
{
60+
"login": "diegomarquezp"
61+
},
62+
{
63+
"login": "distroless-bot"
64+
},
65+
{
66+
"login": "donmccasland"
67+
},
68+
{
69+
"login": "droot"
70+
},
71+
{
72+
"login": "ericzzzzzzz"
73+
},
74+
{
75+
"login": "etefera"
76+
},
77+
{
78+
"login": "gcp-runtimes-bot"
79+
},
80+
{
81+
"login": "google-admin"
82+
},
83+
{
84+
"login": "google-ospo-team"
85+
},
86+
{
87+
"login": "googlebot"
88+
},
89+
{
90+
"login": "gsquared94"
91+
},
92+
{
93+
"login": "haiyanmeng"
94+
},
95+
{
96+
"login": "henrybell"
97+
},
98+
{
99+
"login": "inferno-chromium"
100+
},
101+
{
102+
"login": "janetkuo"
103+
},
104+
{
105+
"login": "jduncan-rva"
106+
},
107+
{
108+
"login": "JeromeJu"
109+
},
110+
{
111+
"login": "jinseopkim0"
112+
},
113+
{
114+
"login": "JoeWang1127"
115+
},
116+
{
117+
"login": "johnbelamaric"
118+
},
119+
{
120+
"login": "joycebrum"
121+
},
122+
{
123+
"login": "justinsb"
124+
},
125+
{
126+
"login": "karlkfi"
127+
},
128+
{
129+
"login": "katiexzhang"
130+
},
131+
{
132+
"login": "kmaydeo"
133+
},
134+
{
135+
"login": "kokoro-team"
136+
},
137+
{
138+
"login": "ldetmer"
139+
},
140+
{
141+
"login": "linde"
142+
},
143+
{
144+
"login": "loosebazooka"
145+
},
146+
{
147+
"login": "loudej"
148+
},
149+
{
150+
"login": "louisjimenez"
151+
},
152+
{
153+
"login": "lqiu96"
154+
},
155+
{
156+
"login": "maggieneterval"
157+
},
158+
{
159+
"login": "MarlonGamez"
160+
},
161+
{
162+
"login": "matthewmichihara"
163+
},
164+
{
165+
"login": "mattsanta"
166+
},
167+
{
168+
"login": "medyagh"
169+
},
170+
{
171+
"login": "meltsufin"
172+
},
173+
{
174+
"login": "menahyouyeah"
175+
},
176+
{
177+
"login": "mikebz"
178+
},
179+
{
180+
"login": "minikube-bot"
181+
},
182+
{
183+
"login": "mortent"
184+
},
185+
{
186+
"login": "mpeddada1"
187+
},
188+
{
189+
"login": "nan-yu"
190+
},
191+
{
192+
"login": "natasha41575"
193+
},
194+
{
195+
"login": "plumpy"
196+
},
197+
{
198+
"login": "renzodavid9"
199+
},
200+
{
201+
"login": "rquitales"
202+
},
203+
{
204+
"login": "sdowell"
205+
},
206+
{
207+
"login": "seans3"
208+
},
209+
{
210+
"login": "spencersugarman"
211+
},
212+
{
213+
"login": "spowelljr"
214+
},
215+
{
216+
"login": "suztomo"
217+
},
218+
{
219+
"login": "SvenMM"
220+
},
221+
{
222+
"login": "tiffanny29631"
223+
},
224+
{
225+
"login": "Yongxuanzhang"
226+
},
227+
{
228+
"login": "yuwenma"
229+
},
230+
{
231+
"login": "zhumin8"
232+
}
233+
]
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
name: 'multi-approvers'
2+
3+
on:
4+
pull_request:
5+
types:
6+
- 'opened'
7+
- 'edited'
8+
- 'reopened'
9+
- 'synchronize'
10+
- 'ready_for_review'
11+
- 'review_requested'
12+
- 'review_request_removed'
13+
pull_request_review:
14+
types:
15+
- 'submitted'
16+
- 'dismissed'
17+
18+
permissions:
19+
actions: 'write'
20+
contents: 'read'
21+
pull-requests: 'read'
22+
23+
concurrency:
24+
group: '${{ github.workflow }}-${{ github.head_ref || github.ref }}'
25+
cancel-in-progress: true
26+
27+
jobs:
28+
multi-approvers:
29+
uses: 'abcxyz/pkg/.github/workflows/multi-approvers.yml@main'
30+
with:
31+
org-members-path: '.github/workflows/members.json'

0 commit comments

Comments
 (0)