Skip to content
This repository was archived by the owner on Jul 1, 2024. It is now read-only.

Commit 7264be7

Browse files
authored
Merge pull request #436 from DefinitelyTyped/github-actions-triage
Put github-actions PRs into NMA
2 parents 6b659eb + caecd39 commit 7264be7

File tree

13 files changed

+960
-3
lines changed

13 files changed

+960
-3
lines changed
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
{}
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
{}
Lines changed: 325 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,325 @@
1+
{
2+
"data": {
3+
"repository": {
4+
"id": "MDEwOlJlcG9zaXRvcnk2MDkzMzE2",
5+
"pullRequest": {
6+
"id": "PR_kwDOAFz6BM4yI35U",
7+
"title": "Remove contributors with deleted accounts",
8+
"createdAt": "2022-02-07T00:04:01Z",
9+
"author": {
10+
"login": "github-actions",
11+
"__typename": "Bot"
12+
},
13+
"authorAssociation": "CONTRIBUTOR",
14+
"baseRef": {
15+
"name": "master",
16+
"__typename": "Ref"
17+
},
18+
"labels": {
19+
"nodes": [
20+
{
21+
"name": "Popular package",
22+
"__typename": "Label"
23+
},
24+
{
25+
"name": "Owner Approved",
26+
"__typename": "Label"
27+
},
28+
{
29+
"name": "Edits multiple packages",
30+
"__typename": "Label"
31+
},
32+
{
33+
"name": "Where is GH Actions?",
34+
"__typename": "Label"
35+
},
36+
{
37+
"name": "Untested Change",
38+
"__typename": "Label"
39+
},
40+
{
41+
"name": "Edits Owners",
42+
"__typename": "Label"
43+
}
44+
],
45+
"__typename": "LabelConnection"
46+
},
47+
"isDraft": false,
48+
"mergeable": "UNKNOWN",
49+
"number": 58632,
50+
"state": "MERGED",
51+
"headRefOid": "d7304db2a17425c0b41451f5c2e056d063725ecf",
52+
"changedFiles": 6,
53+
"additions": 8,
54+
"deletions": 12,
55+
"commitIds": {
56+
"nodes": [
57+
{
58+
"commit": {
59+
"oid": "d7304db2a17425c0b41451f5c2e056d063725ecf",
60+
"parents": {
61+
"nodes": [
62+
{
63+
"oid": "e33d3fe8b10acee209c6353529cf180b3cac25ac",
64+
"__typename": "Commit"
65+
}
66+
],
67+
"__typename": "CommitConnection"
68+
},
69+
"__typename": "Commit"
70+
},
71+
"__typename": "PullRequestCommit"
72+
}
73+
],
74+
"__typename": "PullRequestCommitConnection"
75+
},
76+
"timelineItems": {
77+
"nodes": [],
78+
"__typename": "PullRequestTimelineItemsConnection"
79+
},
80+
"reviews": {
81+
"nodes": [
82+
{
83+
"author": {
84+
"login": "alitaheri",
85+
"__typename": "User"
86+
},
87+
"commit": {
88+
"oid": "d7304db2a17425c0b41451f5c2e056d063725ecf",
89+
"__typename": "Commit"
90+
},
91+
"comments": {
92+
"nodes": [],
93+
"__typename": "PullRequestReviewCommentConnection"
94+
},
95+
"authorAssociation": "CONTRIBUTOR",
96+
"state": "APPROVED",
97+
"submittedAt": "2022-02-08T14:24:13Z",
98+
"url": "https://github.com/DefinitelyTyped/DefinitelyTyped/pull/58632#pullrequestreview-876105577",
99+
"__typename": "PullRequestReview"
100+
},
101+
{
102+
"author": {
103+
"login": "candyapplecorn",
104+
"__typename": "User"
105+
},
106+
"commit": {
107+
"oid": "d7304db2a17425c0b41451f5c2e056d063725ecf",
108+
"__typename": "Commit"
109+
},
110+
"comments": {
111+
"nodes": [],
112+
"__typename": "PullRequestReviewCommentConnection"
113+
},
114+
"authorAssociation": "CONTRIBUTOR",
115+
"state": "APPROVED",
116+
"submittedAt": "2022-02-09T22:06:31Z",
117+
"url": "https://github.com/DefinitelyTyped/DefinitelyTyped/pull/58632#pullrequestreview-878109819",
118+
"__typename": "PullRequestReview"
119+
},
120+
{
121+
"author": {
122+
"login": "al-fyodorov",
123+
"__typename": "User"
124+
},
125+
"commit": {
126+
"oid": "d7304db2a17425c0b41451f5c2e056d063725ecf",
127+
"__typename": "Commit"
128+
},
129+
"comments": {
130+
"nodes": [],
131+
"__typename": "PullRequestReviewCommentConnection"
132+
},
133+
"authorAssociation": "CONTRIBUTOR",
134+
"state": "APPROVED",
135+
"submittedAt": "2022-02-15T11:39:02Z",
136+
"url": "https://github.com/DefinitelyTyped/DefinitelyTyped/pull/58632#pullrequestreview-882856991",
137+
"__typename": "PullRequestReview"
138+
}
139+
],
140+
"__typename": "PullRequestReviewConnection"
141+
},
142+
"commits": {
143+
"totalCount": 1,
144+
"nodes": [
145+
{
146+
"commit": {
147+
"checkSuites": {
148+
"nodes": [
149+
{
150+
"databaseId": 5197815169,
151+
"app": {
152+
"name": "Azure Pipelines",
153+
"__typename": "App"
154+
},
155+
"conclusion": "SUCCESS",
156+
"resourcePath": "/DefinitelyTyped/DefinitelyTyped/commit/d7304db2a17425c0b41451f5c2e056d063725ecf/checks?check_suite_id=5197815169",
157+
"status": "COMPLETED",
158+
"url": "https://github.com/DefinitelyTyped/DefinitelyTyped/commit/d7304db2a17425c0b41451f5c2e056d063725ecf/checks?check_suite_id=5197815169",
159+
"checkRuns": {
160+
"nodes": [
161+
{
162+
"title": "Build #20220207.1 succeeded",
163+
"__typename": "CheckRun"
164+
}
165+
],
166+
"__typename": "CheckRunConnection"
167+
},
168+
"__typename": "CheckSuite"
169+
},
170+
{
171+
"databaseId": 5197815175,
172+
"app": {
173+
"name": "Dependabot",
174+
"__typename": "App"
175+
},
176+
"conclusion": null,
177+
"resourcePath": "/DefinitelyTyped/DefinitelyTyped/commit/d7304db2a17425c0b41451f5c2e056d063725ecf/checks?check_suite_id=5197815175",
178+
"status": "QUEUED",
179+
"url": "https://github.com/DefinitelyTyped/DefinitelyTyped/commit/d7304db2a17425c0b41451f5c2e056d063725ecf/checks?check_suite_id=5197815175",
180+
"checkRuns": {
181+
"nodes": [],
182+
"__typename": "CheckRunConnection"
183+
},
184+
"__typename": "CheckSuite"
185+
}
186+
],
187+
"__typename": "CheckSuiteConnection"
188+
},
189+
"status": null,
190+
"authoredDate": "2022-02-07T00:03:58Z",
191+
"committedDate": "2022-02-07T00:03:58Z",
192+
"pushedDate": "2022-02-07T00:04:05Z",
193+
"oid": "d7304db2a17425c0b41451f5c2e056d063725ecf",
194+
"__typename": "Commit"
195+
},
196+
"__typename": "PullRequestCommit"
197+
}
198+
],
199+
"__typename": "PullRequestCommitConnection"
200+
},
201+
"comments": {
202+
"totalCount": 4,
203+
"nodes": [
204+
{
205+
"id": "IC_kwDOAFz6BM49cv4g",
206+
"author": {
207+
"login": "typescript-bot",
208+
"__typename": "User"
209+
},
210+
"databaseId": 1030946336,
211+
"body": "@github-actions Thank you for submitting this PR!\n\n***This is a live comment which I will keep updated.***\n\n## 6 packages in this PR\n\n* `mithril-global` — [on npm](https://www.npmjs.com/package/mithril-global), [on unpkg](https://unpkg.com/browse/mithril-global@latest/)\n - 1 removed owner: @isiahmeadows\n* `mithril` — [on npm](https://www.npmjs.com/package/mithril), [on unpkg](https://unpkg.com/browse/mithril@latest/)\n - 1 removed owner: @isiahmeadows\n* `ospec` — [on npm](https://www.npmjs.com/package/ospec), [on unpkg](https://unpkg.com/browse/ospec@latest/)\n - 1 removed owner: @isiahmeadows\n* `react-grid-layout` — [on npm](https://www.npmjs.com/package/react-grid-layout), [on unpkg](https://unpkg.com/browse/react-grid-layout@latest/)\n - owner-approval: @al-fyodorov, @alitaheri\n - 1 removed owner: @catusmagnus\n* `react-native-fbsdk` — [on npm](https://www.npmjs.com/package/react-native-fbsdk), [on unpkg](https://unpkg.com/browse/react-native-fbsdk@latest/)\n - 1 removed owner: @UHKim\n* `ssh2-sftp-client` — [on npm](https://www.npmjs.com/package/ssh2-sftp-client), [on unpkg](https://unpkg.com/browse/ssh2-sftp-client@latest/)\n - owner-approval: @candyapplecorn\n - 1 removed owner: @sgalizia\n\n## Code Reviews\n\nBecause this PR edits multiple packages, it can be merged once it's reviewed by a DT maintainer.\n\nYou can test the changes of this PR [in the Playground](https://www.typescriptlang.org/play/?dtPR=58632&install-plugin=playground-dt-review).\n\n## Status\n\n * ✅ No merge conflicts\n * ❓ Continuous integration tests have gone missing\n * 🕐 A DT maintainer needs to approve changes which affect more than one package\n\nOnce every item on this list is checked, I'll ask you for permission to merge and publish the changes.\n\n## Inactive\n\nThis PR has been inactive for 10 days.\n\n----------------------\n... diagnostics scrubbed ...\n<!--typescript_bot_welcome-->",
212+
"createdAt": "2022-02-07T00:04:41Z",
213+
"reactions": {
214+
"nodes": [],
215+
"__typename": "ReactionConnection"
216+
},
217+
"__typename": "IssueComment"
218+
},
219+
{
220+
"id": "IC_kwDOAFz6BM49cv4q",
221+
"author": {
222+
"login": "typescript-bot",
223+
"__typename": "User"
224+
},
225+
"databaseId": 1030946346,
226+
"body": "🔔 @spacejack @isiahmeadows @maranomynet @abirkholz @alitaheri @ZheyangSong @andrewhathaway @manav-m @al-fyodorov @catusmagnus @ifiokjr @titozzz @stuartforrest-infinity @UHKim @igrayson @ascariandrea @kartik2406 @viamuli @gbhmt @builtbylane @loru88 @sgalizia @hengkx @candyapplecorn @tsop14 — please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/58632/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on.\n<!--typescript_bot_pinging-reviewers-->",
227+
"createdAt": "2022-02-07T00:04:43Z",
228+
"reactions": {
229+
"nodes": [],
230+
"__typename": "ReactionConnection"
231+
},
232+
"__typename": "IssueComment"
233+
},
234+
{
235+
"id": "IC_kwDOAFz6BM49pTyu",
236+
"author": {
237+
"login": "candyapplecorn",
238+
"__typename": "User"
239+
},
240+
"databaseId": 1034239150,
241+
"body": "This doesn't seem morally right. I don't know how to remove my approval, but if I could I would.\r\nJust because their accounts were deleted, doesn't invalidate their past work.\r\nEdit - Unless the purpose of putting ones' name is to let other contributors have people to contact for info/help? But in my experience, it's always been just as much about getting one's name on a contributor list, as it is about recording a breadcrumb trail of past contributors for new contributors to access as a resource.",
242+
"createdAt": "2022-02-09T22:08:45Z",
243+
"reactions": {
244+
"nodes": [],
245+
"__typename": "ReactionConnection"
246+
},
247+
"__typename": "IssueComment"
248+
},
249+
{
250+
"id": "IC_kwDOAFz6BM4-MZQM",
251+
"author": {
252+
"login": "andrewbranch",
253+
"__typename": "User"
254+
},
255+
"databaseId": 1043436556,
256+
"body": "@candyapplecorn the purpose of the header comment is not attribution; it’s ongoing maintenance. These accounts get automatically pinged for reviews on future PRs (like you were pinged here), and when they approve, the PRs get fast-tracked to merge. The reason we have to delete inactive ones is that GitHub releases the usernames of deleted accounts after a while, so new users could be signing up and be getting pinged for reviews of something they’ve never heard of. I’m sympathetic to the unintended consequence of erasing something that someone wanted attribution for, but the reality is that people intentionally remove themselves from these headers all the time when they want to stop getting pinged for reviews, and we have no way of continuing to list them as contributors either, other than in the historical record (both in the git history and in the published package.jsons)—that’s just not what the header was made for.",
257+
"createdAt": "2022-02-17T21:00:53Z",
258+
"reactions": {
259+
"nodes": [],
260+
"__typename": "ReactionConnection"
261+
},
262+
"__typename": "IssueComment"
263+
}
264+
],
265+
"__typename": "IssueCommentConnection"
266+
},
267+
"files": {
268+
"totalCount": 6,
269+
"nodes": [
270+
{
271+
"path": "types/mithril-global/index.d.ts",
272+
"additions": 1,
273+
"deletions": 1,
274+
"__typename": "PullRequestChangedFile"
275+
},
276+
{
277+
"path": "types/mithril/index.d.ts",
278+
"additions": 1,
279+
"deletions": 1,
280+
"__typename": "PullRequestChangedFile"
281+
},
282+
{
283+
"path": "types/ospec/index.d.ts",
284+
"additions": 0,
285+
"deletions": 1,
286+
"__typename": "PullRequestChangedFile"
287+
},
288+
{
289+
"path": "types/react-grid-layout/index.d.ts",
290+
"additions": 6,
291+
"deletions": 7,
292+
"__typename": "PullRequestChangedFile"
293+
},
294+
{
295+
"path": "types/react-native-fbsdk/index.d.ts",
296+
"additions": 0,
297+
"deletions": 1,
298+
"__typename": "PullRequestChangedFile"
299+
},
300+
{
301+
"path": "types/ssh2-sftp-client/index.d.ts",
302+
"additions": 0,
303+
"deletions": 1,
304+
"__typename": "PullRequestChangedFile"
305+
}
306+
],
307+
"pageInfo": {
308+
"hasNextPage": false,
309+
"endCursor": "Ng",
310+
"__typename": "PageInfo"
311+
},
312+
"__typename": "PullRequestChangedFileConnection"
313+
},
314+
"projectCards": {
315+
"nodes": [],
316+
"__typename": "ProjectCardConnection"
317+
},
318+
"__typename": "PullRequest"
319+
},
320+
"__typename": "Repository"
321+
}
322+
},
323+
"loading": false,
324+
"networkStatus": 7
325+
}
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
{
2+
"type": "remove",
3+
"now": "2022-02-17T22:40:31.072Z",
4+
"message": "PR is not active",
5+
"isDraft": false
6+
}
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
[]
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
{
2+
"labels": [],
3+
"responseComments": [],
4+
"shouldClose": false,
5+
"shouldMerge": false,
6+
"shouldUpdateLabels": false,
7+
"projectColumn": "*REMOVE*"
8+
}
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
{
2+
"bootstrap.v3.datetimepicker": 2109,
3+
"mithril-global": 23,
4+
"mithril": 15949,
5+
"ospec": 791,
6+
"react-grid-layout": 520136,
7+
"react-native-fbsdk": 78391,
8+
"react-virtualized": 1521648,
9+
"ssh2-sftp-client": 307276,
10+
"styled-components": 8672781
11+
}

0 commit comments

Comments
 (0)