only use node-fetch when needed

microsoft · Sep 18, 2023 · c83e728 · c83e728
1 parent b297d96
commit c83e728
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/incubator/build/src/remotes/github.ts b/incubator/build/src/remotes/github.ts
@@ -36,7 +36,12 @@ const workflowRunCache: Record<string, number> = {};
 
 const octokit = once(() => {
   const RestClient = Octokit.plugin(restEndpointMethods);
-  return new RestClient({ auth: getPersonalAccessToken(), request: { fetch } });
+  return new RestClient({
+    auth: getPersonalAccessToken(),
+    // Use `node-fetch` only if Node doesn't have it:
+    // https://github.com/octokit/request.js/blob/v8.1.1/src/fetch-wrapper.ts#L28-L31
+    request: "fetch" in globalThis ? undefined : { fetch },
+  });
 });
 
 async function downloadArtifact(