Remove Participants related codes for pr view`

doi-t · doi-t · commit 6d0e5bf2b8d6 · 2020-04-10T01:02:51.000+09:00
diff --git a/api/queries_pr.go b/api/queries_pr.go
@@ -107,12 +107,6 @@ type PullRequest struct {
 	Milestone struct {
 		Title string
 	}
-	Participants struct {
-		Nodes []struct {
-			Login string
-		}
-		TotalCount int
-	}
 }
 
 type NotFoundError struct {
@@ -423,12 +417,6 @@ func PullRequestByNumber(client *Client, repo ghrepo.Interface, number int) (*Pu
 				milestone{
 					title
 				}
-				participants(first: 100) {
-					nodes {
-						login
-					}
-					totalCount
-				}
 			}
 		}
 	}`
@@ -528,12 +516,6 @@ func PullRequestForBranch(client *Client, repo ghrepo.Interface, baseBranch, hea
 					milestone{
 						title
 					}
-					participants(first: 100) {
-						nodes {
-							login
-						}
-						totalCount
-					}
 				}
 			}
 		}
diff --git a/command/pr.go b/command/pr.go
@@ -425,23 +425,6 @@ func prProjectList(pr api.PullRequest) string {
 	return list
 }
 
-func prParticipantList(pr api.PullRequest) string {
-	if len(pr.Participants.Nodes) == 0 {
-		return ""
-	}
-
-	participantNames := make([]string, 0, len(pr.Participants.Nodes))
-	for _, participant := range pr.Participants.Nodes {
-		participantNames = append(participantNames, participant.Login)
-	}
-
-	list := strings.Join(participantNames, ", ")
-	if pr.Participants.TotalCount > len(pr.Participants.Nodes) {
-		list += ", …"
-	}
-	return list
-}
-
 var prURLRE = regexp.MustCompile(`^https://github\.com/([^/]+)/([^/]+)/pull/(\d+)`)
 
 func prFromURL(arg string) (string, ghrepo.Interface) {
diff --git a/test/fixtures/prViewPreview.json b/test/fixtures/prViewPreview.json
@@ -25,14 +25,6 @@
         "milestone": {
           "title": ""
         },
-        "participants": {
-          "nodes": [
-            {
-              "login": "marseilles"
-            }
-          ],
-          "totalcount": 1
-        },
         "commits": {
           "totalCount": 12
         },
diff --git a/test/fixtures/prViewPreviewWithMetadataByBranch.json b/test/fixtures/prViewPreviewWithMetadataByBranch.json
@@ -27,14 +27,6 @@
               "totalcount": 0
             },
             "milestone": {},
-            "participants": {
-              "nodes": [
-                {
-                  "login": "nobody"
-                }
-              ],
-              "totalcount": 1
-            },
             "commits": {
               "totalCount": 12
             },
@@ -118,20 +110,6 @@
             "milestone": {
               "title": "uluru"
             },
-            "participants": {
-              "nodes": [
-                {
-                  "login": "marseilles"
-                },
-                {
-                  "login": "monaco"
-                },
-                {
-                  "login": "montpellier"
-                }
-              ],
-              "totalcount": 3
-            },
             "headRepositoryOwner": {
               "login": "OWNER"
             },
diff --git a/test/fixtures/prViewPreviewWithMetadataByNumber.json b/test/fixtures/prViewPreviewWithMetadataByNumber.json
@@ -73,20 +73,6 @@
         "milestone": {
           "title": "uluru"
         },
-        "participants": {
-          "nodes": [
-            {
-              "login": "marseilles"
-            },
-            {
-              "login": "monaco"
-            },
-            {
-              "login": "montpellier"
-            }
-          ],
-          "totalcount": 3
-        },
         "commits": {
           "totalCount": 12
         },

Original file line number	Diff line number	Diff line change
`@@ -107,12 +107,6 @@ type PullRequest struct {`
`107`	`107`	`Milestone struct {`
`108`	`108`	`Title string`
`109`	`109`	`}`
`110`		`- Participants struct {`
`111`		`- Nodes []struct {`
`112`		`- Login string`
`113`		`- }`
`114`		`- TotalCount int`
`115`		`- }`
`116`	`110`	`}`
`117`	`111`
`118`	`112`	`type NotFoundError struct {`
`@@ -423,12 +417,6 @@ func PullRequestByNumber(client Client, repo ghrepo.Interface, number int) (Pu`
`423`	`417`	`milestone{`
`424`	`418`	`title`
`425`	`419`	`}`
`426`		`- participants(first: 100) {`
`427`		`- nodes {`
`428`		`- login`
`429`		`- }`
`430`		`- totalCount`
`431`		`- }`
`432`	`420`	`}`
`433`	`421`	`}`
`434`	`422`	}`
`@@ -528,12 +516,6 @@ func PullRequestForBranch(client *Client, repo ghrepo.Interface, baseBranch, hea`
`528`	`516`	`milestone{`
`529`	`517`	`title`
`530`	`518`	`}`
`531`		`- participants(first: 100) {`
`532`		`- nodes {`
`533`		`- login`
`534`		`- }`
`535`		`- totalCount`
`536`		`- }`
`537`	`519`	`}`
`538`	`520`	`}`
`539`	`521`	`}`