cleanup

intro-skipper · Sep 30, 2024 · 58d7c71 · 58d7c71
1 parent 24fba2e
commit 58d7c71
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/validate-and-update-manifest.ts b/src/validate-and-update-manifest.ts
@@ -8,7 +8,7 @@ import { URL } from 'url'
 const repository = process.env.GITHUB_REPOSITORY
 const version = process.env.NEW_FILE_VERSION
 const gitHubRepoVisibilty = process.env.GITHUB_REPO_VISIBILITY
-let currentVersion: string | undefined
+let currentVersion: string
 let targetAbi = ''
 
 type Manifest = {
@@ -286,13 +286,14 @@ export function filterVersions(
     const latestVersion = matchingVersions[matchingVersions.length - 1]
     return latestVersion
   }
+  core.setFailed(`No versions match the pattern ${versionPattern}`)
   return undefined
 }
 
 async function getNugetPackageVersion(
   packageName: string,
   versionPattern: string
-): Promise<string | undefined> {
+): Promise<string> {
   try {
     const versions = await fetchNugetPackageVersions(packageName)
     const latestVersion = filterVersions(versions, versionPattern)
@@ -310,5 +311,6 @@ async function getNugetPackageVersion(
   } catch (error) {
     core.setFailed(String(error))
   }
-  return undefined
+  core.setFailed(`Something went wrong while fetching ${packageName}`)
+  return ''
 }