Add check for published packages before returning status skipped

jill64 · Dec 20, 2023 · 70f10e5 · 70f10e5
1 parent 74ed5e9
commit 70f10e5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/packages/action/src/ghosts/release/index.ts b/packages/action/src/ghosts/release/index.ts
@@ -26,7 +26,14 @@ export const release: Ghost = async ({ payload: { owner, repo }, octokit }) => {
     }
   }
 
-  await Promise.allSettled(files.map(npmPublish))
+  const result = await Promise.allSettled(files.map(npmPublish))
+
+  if (!result.some((r) => r.status === 'fulfilled' && r.value)) {
+    return {
+      status: 'skipped',
+      detail: 'No package published'
+    }
+  }
 
   const json = await getPackageJson()
   const rootPackageJson = isValidPackageJson(json) ? json : null

diff --git a/packages/action/src/ghosts/release/npmPublish.ts b/packages/action/src/ghosts/release/npmPublish.ts
@@ -18,7 +18,7 @@ export const npmPublish = async (file: string) => {
 
   if (!isValidJson(package_json)) {
     core.info(`[${file}]: No version found.`)
-    return
+    return false
   }
 
   const version = package_json.version.trim()
@@ -34,10 +34,12 @@ export const npmPublish = async (file: string) => {
 
   if (version === publishedVersion.stdout.trim()) {
     core.info(`[${file}]: No update found.`)
-    return
+    return false
   }
 
   await exec.exec('npm publish', undefined, {
     cwd
   })
+
+  return true
 }