Introduce a type for GitHub issue comments

danger · Dec 27, 2018 · dbc9615 · dbc9615
1 parent 0eb5b4c
commit dbc9615
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 9 deletions.
diff --git a/source/dsl/GitHubDSL.ts b/source/dsl/GitHubDSL.ts
@@ -136,6 +136,23 @@ export interface GitHubIssueLabel {
   color: string
 }
 
+export interface GitHubIssueComment {
+  /**
+   *  UUID for the comment
+   */
+  id: string
+
+  /**
+   * The User who made the comment
+   */
+  user: GitHubUser
+
+  /**
+   * Textual representation of comment
+   */
+  body: string
+}
+
 // This is `danger.github.pr`
 
 /**

diff --git a/source/platforms/github/GitHubAPI.ts b/source/platforms/github/GitHubAPI.ts
@@ -4,11 +4,10 @@ import * as node_fetch from "node-fetch"
 import parse from "parse-link-header"
 import pLimit from "p-limit"
 
-import { GitHubPRDSL, GitHubUser } from "../../dsl/GitHubDSL"
+import { GitHubPRDSL, GitHubIssueComment, GitHubUser } from "../../dsl/GitHubDSL"
 
 import { dangerIDToString } from "../../runner/templates/githubIssueTemplate"
 import { api as fetch } from "../../api/fetch"
-import { Comment } from "../platform"
 import { RepoMetaData } from "../../dsl/BitBucketServerDSL"
 import { CheckOptions } from "./comms/checks/resultsToCheck"
 
@@ -85,7 +84,7 @@ export class GitHubAPI {
 
   getDangerCommentIDs = async (dangerID: string): Promise<string[]> => {
     const userID = await this.getUserID()
-    const allComments: any[] = await this.getPullRequestComments()
+    const allComments = await this.getPullRequestComments()
     const dangerIDMessage = dangerIDToString(dangerID)
     this.d(`User ID: ${userID}`)
     this.d(`Looking at ${allComments.length} comments for ${dangerIDMessage}`)
@@ -274,23 +273,23 @@ export class GitHubAPI {
     return response.json()
   }
 
-  getPullRequestComments = async (): Promise<any[]> => {
+  getPullRequestComments = async (): Promise<GitHubIssueComment[]> => {
     const repo = this.repoMetadata.repoSlug
     const prID = this.repoMetadata.pullRequestID
     return await this.getAllOfResource(`repos/${repo}/issues/${prID}/comments`)
   }
 
-  getPullRequestInlineComments = async (dangerID: string): Promise<Comment[]> => {
+  getPullRequestInlineComments = async (dangerID: string): Promise<(GitHubIssueComment & { ownedByDanger: boolean })[]> => {
     const userID = await this.getUserID()
     const repo = this.repoMetadata.repoSlug
     const prID = this.repoMetadata.pullRequestID
-    return await this.getAllOfResource(`repos/${repo}/pulls/${prID}/comments`).then(v => {
+    return await this.getAllOfResource(`repos/${repo}/pulls/${prID}/comments`).then((v: GitHubIssueComment[]) => {
       return v
         .filter(Boolean)
-        .map((i: any) => {
-          return { id: i.id, ownedByDanger: i.user.id == userID && i.body.includes(dangerID), body: i.body }
+        .map(i => {
+          return { ...i, ownedByDanger: i.user.id == userID && i.body.includes(dangerID) }
         })
-        .filter((i: any) => i.ownedByDanger)
+        .filter(i => i.ownedByDanger)
     })
   }