Fix JS syntax error

It seems that the `foo?.bar` syntax is not compatible with whatever CJS loader Actions is using. Fixes #16
mislav · Mar 2, 2021 · 2958320 · 2958320
1 parent b7633ec
commit 2958320
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/api.ts b/src/api.ts
@@ -9,9 +9,9 @@ const GitHub = Octokit.plugin(restEndpointMethods, requestLog).defaults({
 
 export type API = InstanceType<typeof GitHub>
 
-export default function (token: string, options?: {fetch?: any}): API {
+export default function (token: string, options?: { fetch?: any }): API {
   return new GitHub({
-    request: {fetch: options?.fetch},
+    request: { fetch: options && options.fetch },
     auth: `token ${token}`,
     log: {
       info(msg: string) {

diff --git a/src/edit-github-blob.ts b/src/edit-github-blob.ts
@@ -47,7 +47,8 @@ export default async function (params: Options): Promise<string> {
     branch: baseBranch,
   })
 
-  const needsFork = !repoRes.data.permissions?.push
+  const needsFork =
+    repoRes.data.permissions == null || !repoRes.data.permissions.push
   if (needsFork) {
     const res = await Promise.all([
       api.repos.createFork(baseRepo),