Apply suggestions from code review

Co-authored-by: Henry Mercer <henry.mercer@me.com>
github · Jun 13, 2023 · fa9ebea · fa9ebea
1 parent 7c5757a
commit fa9ebea
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 6 deletions.
diff --git a/pr-checks/checks/resolve-environment-action.yml b/pr-checks/checks/resolve-environment-action.yml
@@ -1,4 +1,4 @@
-name: "resolve-environment-action"
+name: "Resolve environment"
 description: "Tests that the resolve-environment action works for Go"
 versions: ["nightly-latest"]
 steps:

diff --git a/src/codeql.ts b/src/codeql.ts
@@ -239,7 +239,6 @@ export interface ResolveQueriesOutput {
 export interface ResolveBuildEnvironmentOutput {
   configuration?: {
     [language: string]: {
-      os?: unknown;
       [key: string]: unknown;
     };
   };

diff --git a/src/resolve-environment-action.ts b/src/resolve-environment-action.ts
@@ -58,11 +58,11 @@ async function run() {
     core.setOutput("environment", result);
   } catch (unwrappedError) {
     const error = wrapError(unwrappedError);
-    core.setFailed(error.message);
+    core.setFailed(`Failed to resolve a build environment suitable for automatically building your code. ${error.message}`);
     await sendStatusReport(
       await createStatusReportBase(
         ACTION_NAME,
-        "aborted",
+        getActionsStatus(error),
         startedAt,
         error.message,
         error.stack

diff --git a/src/resolve-environment.ts b/src/resolve-environment.ts
@@ -11,8 +11,8 @@ export async function runResolveBuildEnvironment(
   logger.startGroup(
     `Attempting to resolve build environment for ${language} in ${workingDir}`
   );
-  const codeQL = await getCodeQL(cmd);
-  const result = await codeQL.resolveBuildEnvironment(workingDir, language);
+  const codeql = await getCodeQL(cmd);
+  const result = await codeql.resolveBuildEnvironment(workingDir, language);
   logger.endGroup();
   return result;
 }