do not include required_contexts at all

Author: gjtorikian

src/lib/context.ts

@@ -3,7 +3,6 @@ import {
   getBooleanInput,
   getOptionalInput,
   getRequiredInput,
-  parseOptionalStringArrayInput,
 } from "./input";
 import Logger from "./log";
 

src/steps/index.ts

@@ -37,14 +37,17 @@ export async function run(
           if (rawPayload) {
             payload = JSON.parse(rawPayload);
           }
-          const stepArgs: StartArgs = {
+          let start_args = {
             deploymentID: getOptionalInput("deployment_id"),
             override: getBooleanInput("override", false), // default to false on start
             auto_merge: getBooleanInput("auto_merge", false),
-            required_contexts:
-              parseOptionalStringArrayInput("required_contexts"),
             payload,
           };
+          const required_contexts = parseOptionalStringArrayInput("required_contexts")
+          if (required_contexts !== undefined) {
+            start_args["required_contexts"] = required_contexts;
+          };
+          const stepArgs: StartArgs = start_args;
           log.debug(`'${step}' arguments`, {
             stepArgs,
             coreArgs,

src/steps/start.ts

@@ -7,7 +7,7 @@ export type StartArgs = {
   deploymentID?: string;
   override: boolean;
   auto_merge: boolean;
-  required_contexts: string[] | undefined;
+  required_contexts?: string[];
   payload?: { [key: string]: any };
 };