Explicitly declare return types (#1589)

Co-authored-by: Ryan Ling <[email protected]>

.changeset/friendly-houses-promise.md

@@ -0,0 +1,5 @@
+---
+'skuba': patch
+---
+
+Git: Explicitly declare return types to enable compatibility with the `Node16` module

src/api/git/pull.ts

@@ -51,7 +51,7 @@ export const fastForwardBranch = async ({
   ref,
   remote,
   remoteRef,
-}: PullParameters) => {
+}: PullParameters): Promise<void> => {
   const { owner, repo } = await getOwnerAndRepo({ dir });
 
   const url = `https://github.com/${encodeURIComponent(

src/api/git/push.ts

@@ -51,6 +51,19 @@ interface PushParameters {
   force?: boolean;
 }
 
+interface PushResult {
+  ok: boolean;
+  error: string | null;
+  refs: Record<
+    string,
+    {
+      ok: boolean;
+      error: string;
+    }
+  >;
+  headers?: Record<string, string> | undefined;
+}
+
 /**
  * Pushes the specified `ref` from the local Git repository to a remote.
  */
@@ -61,7 +74,7 @@ export const push = async ({
   remote,
   remoteRef,
   force,
-}: PushParameters) => {
+}: PushParameters): Promise<PushResult> => {
   const { owner, repo } = await getOwnerAndRepo({ dir });
 
   const url = `https://github.com/${encodeURIComponent(

src/api/github/push.ts

@@ -1,10 +1,6 @@
 import path from 'path';
 
-import type {
-  CreateCommitOnBranchInput,
-  FileAddition,
-  FileDeletion,
-} from '@octokit/graphql-schema';
+import type { CreateCommitOnBranchInput } from '@octokit/graphql-schema';
 import fs from 'fs-extra';
 
 import * as Git from '../git';
@@ -101,6 +97,15 @@ export const uploadAllFileChanges = async ({
   return commitId;
 };
 
+interface FileAddition {
+  contents: unknown;
+  path: string;
+}
+
+interface FileDeletion {
+  path: string;
+}
+
 export interface FileChanges {
   additions: FileAddition[];
   deletions: FileDeletion[];