refactor: use PullRequestApi in PullRequest class

redhat-plumbers-in-action · Nov 8, 2023 · a392372 · a392372
1 parent 56379fd
commit a392372
Show file tree

Hide file tree

Showing 9 changed files with 19 additions and 19 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/dist/pull-request.d.ts b/dist/pull-request.d.ts
diff --git a/dist/pull-request.js.map b/dist/pull-request.js.map
diff --git a/dist/schema/pull-request.d.ts b/dist/schema/pull-request.d.ts
diff --git a/dist/schema/pull-request.js b/dist/schema/pull-request.js
diff --git a/dist/schema/pull-request.js.map b/dist/schema/pull-request.js.map
diff --git a/src/pull-request.ts b/src/pull-request.ts
@@ -9,12 +9,12 @@ export class PullRequest {
   readonly number: number;
   readonly url: string;
 
-  title = '';
-  targetBranch = '';
+  title: PullRequestApi['title'] = '';
+  targetBranch: PullRequestApi['base'] = '';
   mergeable: PullRequestApi['mergeable'] = null;
-  mergeableState = 'unknown';
-  draft: boolean | undefined;
-  currentLabels: string[] = [];
+  mergeableState: PullRequestApi['mergeable_state'] = 'unknown';
+  draft: PullRequestApi['draft'];
+  currentLabels: PullRequestApi['labels'] = [];
 
   constructor(
     metadata: PullRequestMetadata,

diff --git a/src/schema/pull-request.ts b/src/schema/pull-request.ts
@@ -7,7 +7,7 @@ export const pullRequestApiSchema = z.object({
   labels: z.array(
     z.object({ name: z.string() }).transform(label => label.name)
   ),
-  draft: z.boolean(),
+  draft: z.boolean().optional(),
   merged: z.boolean(),
   mergeable: z.boolean().nullable(),
   mergeable_state: z.string(),