Remove pr_number from Actions

jablko · elibarzilay · commit c5dd9910c81d · 2021-01-19T14:06:50.000-05:00
Unused, since `executePrActions` uses IDs that are taken from the PR
query result.

Update snapshots
diff --git a/src/_tests/fixtures/38979/result.json b/src/_tests/fixtures/38979/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 38979,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "Critical package",
diff --git a/src/_tests/fixtures/43136/result.json b/src/_tests/fixtures/43136/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43136,
   "targetColumn": "Needs Maintainer Review",
   "labels": [
     "Critical package"
diff --git a/src/_tests/fixtures/43144/result.json b/src/_tests/fixtures/43144/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43144,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Owner Approved",
diff --git a/src/_tests/fixtures/43151/result.json b/src/_tests/fixtures/43151/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43151,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "New Definition"
diff --git a/src/_tests/fixtures/43160/result.json b/src/_tests/fixtures/43160/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43160,
   "targetColumn": "Needs Author Action",
   "labels": [
     "The CI failed",
diff --git a/src/_tests/fixtures/43175/result.json b/src/_tests/fixtures/43175/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43175,
   "targetColumn": "Needs Maintainer Review",
   "labels": [
     "Popular package",
diff --git a/src/_tests/fixtures/43235/result.json b/src/_tests/fixtures/43235/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43235,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [],
   "responseComments": [
diff --git a/src/_tests/fixtures/43314/result.json b/src/_tests/fixtures/43314/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43314,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "New Definition"
diff --git a/src/_tests/fixtures/43695-duplicate-comment/result.json b/src/_tests/fixtures/43695-duplicate-comment/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43695,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "New Definition",
diff --git a/src/_tests/fixtures/43695-post-review/result.json b/src/_tests/fixtures/43695-post-review/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43695,
   "targetColumn": "Needs Author Action",
   "labels": [
     "Revision needed",
diff --git a/src/_tests/fixtures/43695/result.json b/src/_tests/fixtures/43695/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43695,
   "targetColumn": "Needs Author Action",
   "labels": [
     "Revision needed",
diff --git a/src/_tests/fixtures/43960-post-close/result.json b/src/_tests/fixtures/43960-post-close/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43960,
   "labels": [],
   "responseComments": [],
   "shouldClose": false,
diff --git a/src/_tests/fixtures/43960/result.json b/src/_tests/fixtures/43960/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 43960,
   "targetColumn": "Needs Author Action",
   "labels": [
     "Revision needed",
diff --git a/src/_tests/fixtures/44105/result.json b/src/_tests/fixtures/44105/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44105,
   "labels": [],
   "responseComments": [],
   "shouldClose": false,
diff --git a/src/_tests/fixtures/44256/result.json b/src/_tests/fixtures/44256/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44256,
   "labels": [],
   "responseComments": [],
   "shouldClose": false,
diff --git a/src/_tests/fixtures/44267/result.json b/src/_tests/fixtures/44267/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44267,
   "targetColumn": "Needs Maintainer Review",
   "labels": [
     "Owner Approved",
diff --git a/src/_tests/fixtures/44282/result.json b/src/_tests/fixtures/44282/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44282,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Edits Owners",
diff --git a/src/_tests/fixtures/44288/result.json b/src/_tests/fixtures/44288/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44288,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [],
   "responseComments": [
diff --git a/src/_tests/fixtures/44290/result.json b/src/_tests/fixtures/44290/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44290,
   "labels": [],
   "responseComments": [],
   "shouldClose": false,
diff --git a/src/_tests/fixtures/44299-with-files/result.json b/src/_tests/fixtures/44299-with-files/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44299,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "New Definition"
diff --git a/src/_tests/fixtures/44299/result.json b/src/_tests/fixtures/44299/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44299,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "New Definition"
diff --git a/src/_tests/fixtures/44316/result.json b/src/_tests/fixtures/44316/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44316,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Author is Owner",
diff --git a/src/_tests/fixtures/44343-pending-travis/result.json b/src/_tests/fixtures/44343-pending-travis/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44343,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [],
   "responseComments": [
diff --git a/src/_tests/fixtures/44343-pre-travis/result.json b/src/_tests/fixtures/44343-pre-travis/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44343,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [],
   "responseComments": [
diff --git a/src/_tests/fixtures/44343/result.json b/src/_tests/fixtures/44343/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44343,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [],
   "responseComments": [
diff --git a/src/_tests/fixtures/44402/result.json b/src/_tests/fixtures/44402/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44402,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Other Approved",
diff --git a/src/_tests/fixtures/44411/result.json b/src/_tests/fixtures/44411/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44411,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Edits Owners"
diff --git a/src/_tests/fixtures/44424-1-travis-instantly-finished/result.json b/src/_tests/fixtures/44424-1-travis-instantly-finished/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44424,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Author is Owner"
diff --git a/src/_tests/fixtures/44424-2-after-travis-second/result.json b/src/_tests/fixtures/44424-2-after-travis-second/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44424,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Author is Owner"
diff --git a/src/_tests/fixtures/44437/result.json b/src/_tests/fixtures/44437/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44437,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Popular package",
diff --git a/src/_tests/fixtures/44439/result.json b/src/_tests/fixtures/44439/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44439,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "Edits Owners",
diff --git a/src/_tests/fixtures/44631/result.json b/src/_tests/fixtures/44631/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44631,
   "targetColumn": "Needs Author Action",
   "labels": [
     "Has Merge Conflict",
diff --git a/src/_tests/fixtures/44857/result.json b/src/_tests/fixtures/44857/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44857,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "Critical package",
diff --git a/src/_tests/fixtures/44989-14days/result.json b/src/_tests/fixtures/44989-14days/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44989,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Popular package",
diff --git a/src/_tests/fixtures/44989-32days/result.json b/src/_tests/fixtures/44989-32days/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44989,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Popular package",
diff --git a/src/_tests/fixtures/44989-3days/result.json b/src/_tests/fixtures/44989-3days/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44989,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Popular package",
diff --git a/src/_tests/fixtures/44989-7days/result.json b/src/_tests/fixtures/44989-7days/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 44989,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Popular package",
diff --git a/src/_tests/fixtures/45137/result.json b/src/_tests/fixtures/45137/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45137,
   "targetColumn": "Needs Maintainer Review",
   "labels": [
     "Popular package",
diff --git a/src/_tests/fixtures/45627/result.json b/src/_tests/fixtures/45627/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45627,
   "targetColumn": "Recently Merged",
   "labels": [
     "Critical package",
diff --git a/src/_tests/fixtures/45836/result.json b/src/_tests/fixtures/45836/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45836,
   "targetColumn": "Needs Maintainer Review",
   "labels": [
     "Owner Approved",
diff --git a/src/_tests/fixtures/45884/result.json b/src/_tests/fixtures/45884/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45884,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Owner Approved",
diff --git a/src/_tests/fixtures/45888/result.json b/src/_tests/fixtures/45888/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45888,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Other Approved",
diff --git a/src/_tests/fixtures/45890/result.json b/src/_tests/fixtures/45890/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45890,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "Other Approved",
diff --git a/src/_tests/fixtures/45946/result.json b/src/_tests/fixtures/45946/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45946,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "Edits Infrastructure",
diff --git a/src/_tests/fixtures/45982/result.json b/src/_tests/fixtures/45982/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45982,
   "targetColumn": "Other",
   "labels": [
     "Mergebot Error"
diff --git a/src/_tests/fixtures/45999/result.json b/src/_tests/fixtures/45999/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 45999,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Critical package",
diff --git a/src/_tests/fixtures/46008/result.json b/src/_tests/fixtures/46008/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 46008,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Owner Approved",
diff --git a/src/_tests/fixtures/46019/result.json b/src/_tests/fixtures/46019/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 46019,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Maintainer Approved",
diff --git a/src/_tests/fixtures/46120/result.json b/src/_tests/fixtures/46120/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 46120,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Popular package",
diff --git a/src/_tests/fixtures/46191/result.json b/src/_tests/fixtures/46191/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 46191,
   "targetColumn": "Needs Author Action",
   "labels": [
     "The CI failed",
diff --git a/src/_tests/fixtures/46196/result.json b/src/_tests/fixtures/46196/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 46196,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [],
   "responseComments": [
diff --git a/src/_tests/fixtures/46279/result.json b/src/_tests/fixtures/46279/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 46279,
   "targetColumn": "Needs Maintainer Action",
   "labels": [
     "Author is Owner",
diff --git a/src/_tests/fixtures/46804/result.json b/src/_tests/fixtures/46804/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 46804,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Untested Change"
diff --git a/src/_tests/fixtures/46879/result.json b/src/_tests/fixtures/46879/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 46879,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Popular package"
diff --git a/src/_tests/fixtures/47017-blessed-and-one-owner/result.json b/src/_tests/fixtures/47017-blessed-and-one-owner/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 47017,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Critical package",
diff --git a/src/_tests/fixtures/47017-blessed-and-two-owner/result.json b/src/_tests/fixtures/47017-blessed-and-two-owner/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 47017,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Critical package",
diff --git a/src/_tests/fixtures/47017-blessed/result.json b/src/_tests/fixtures/47017-blessed/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 47017,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [
     "Critical package",
diff --git a/src/_tests/fixtures/47017/result.json b/src/_tests/fixtures/47017/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 47017,
   "targetColumn": "Needs Maintainer Review",
   "labels": [
     "Critical package",
diff --git a/src/_tests/fixtures/48216/result.json b/src/_tests/fixtures/48216/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 48216,
   "targetColumn": "Waiting for Code Reviews",
   "labels": [],
   "responseComments": [
diff --git a/src/_tests/fixtures/48236/result.json b/src/_tests/fixtures/48236/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 48236,
   "targetColumn": "Recently Merged",
   "labels": [
     "Other Approved",
diff --git a/src/_tests/fixtures/48652-merge-offer/result.json b/src/_tests/fixtures/48652-merge-offer/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 48652,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Owner Approved",
diff --git a/src/_tests/fixtures/48652-prereq/result.json b/src/_tests/fixtures/48652-prereq/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 48652,
   "targetColumn": "Waiting for Author to Merge",
   "labels": [
     "Owner Approved",
diff --git a/src/_tests/fixtures/48652-retract-merge-offer-and-prerequest/result.json b/src/_tests/fixtures/48652-retract-merge-offer-and-prerequest/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 48652,
   "targetColumn": "Needs Author Action",
   "labels": [
     "Revision needed",
diff --git a/src/_tests/fixtures/48652-retract-merge-offer/result.json b/src/_tests/fixtures/48652-retract-merge-offer/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 48652,
   "targetColumn": "Needs Author Action",
   "labels": [
     "Revision needed",
diff --git a/src/_tests/fixtures/48708/result.json b/src/_tests/fixtures/48708/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 48708,
   "targetColumn": "Needs Author Action",
   "labels": [
     "Revision needed",
diff --git a/src/_tests/fixtures/48945/result.json b/src/_tests/fixtures/48945/result.json
@@ -1,5 +1,4 @@
 {
-  "pr_number": 48945,
   "targetColumn": "Needs Author Action",
   "labels": [
     "Revision needed",
diff --git a/src/_tests/fixtures/49417/result.json b/src/_tests/fixtures/49417/result.json
diff --git a/src/_tests/fixtures/49548/result.json b/src/_tests/fixtures/49548/result.json
diff --git a/src/_tests/fixtures/49575/result.json b/src/_tests/fixtures/49575/result.json
diff --git a/src/_tests/fixtures/49841/result.json b/src/_tests/fixtures/49841/result.json
diff --git a/src/_tests/fixtures/50429/result.json b/src/_tests/fixtures/50429/result.json
diff --git a/src/_tests/fixtures/50443/result.json b/src/_tests/fixtures/50443/result.json
diff --git a/src/compute-pr-actions.ts b/src/compute-pr-actions.ts

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 38979,`
`3`	`2`	`"targetColumn": "Needs Maintainer Action",`
`4`	`3`	`"labels": [`
`5`	`4`	`"Critical package",`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 43136,`
`3`	`2`	`"targetColumn": "Needs Maintainer Review",`
`4`	`3`	`"labels": [`
`5`	`4`	`"Critical package"`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 43144,`
`3`	`2`	`"targetColumn": "Waiting for Author to Merge",`
`4`	`3`	`"labels": [`
`5`	`4`	`"Owner Approved",`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 43151,`
`3`	`2`	`"targetColumn": "Needs Maintainer Action",`
`4`	`3`	`"labels": [`
`5`	`4`	`"New Definition"`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 43160,`
`3`	`2`	`"targetColumn": "Needs Author Action",`
`4`	`3`	`"labels": [`
`5`	`4`	`"The CI failed",`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 43175,`
`3`	`2`	`"targetColumn": "Needs Maintainer Review",`
`4`	`3`	`"labels": [`
`5`	`4`	`"Popular package",`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 43235,`
`3`	`2`	`"targetColumn": "Waiting for Code Reviews",`
`4`	`3`	`"labels": [],`
`5`	`4`	`"responseComments": [`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 43314,`
`3`	`2`	`"targetColumn": "Needs Maintainer Action",`
`4`	`3`	`"labels": [`
`5`	`4`	`"New Definition"`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`{`
`2`		`- "pr_number": 43695,`
`3`	`2`	`"targetColumn": "Needs Maintainer Action",`
`4`	`3`	`"labels": [`
`5`	`4`	`"New Definition",`