Don't overload results.type

Author: Jskobos

src/action/index.ts

@@ -28,7 +28,7 @@ async function main(): Promise<void> {
     if (results.type === "none") {
       console.log(`\n📦 ${results.package} v${results.version} is already published to NPM`);
     }
-    else if (results.type === "dry-run") {
+    else if (results.dryRun) {
       console.log(`\n📦 ${results.package} v${results.version} successfully built but not published to NPM`);
     }
     else {

src/cli/index.ts

@@ -37,7 +37,7 @@ export async function main(args: string[]): Promise<void> {
         if (results.type === "none") {
           console.log(`\n📦 ${results.package} v${results.version} is already published to NPM`);
         }
-        else if (results.type === "dry-run") {
+        else if (results.dryRun) {
           console.log(`\n📦 ${results.package} v${results.version} successfully built but not published to NPM`);
         }
         else {

src/npm-publish.ts

@@ -25,9 +25,10 @@ export async function npmPublish(opts: Options = {}): Promise<Results> {
 
   let results: Results = {
     package: manifest.name,
-    type: options.dryRun ? "dry-run" : (diff || "none"),
+    type: diff || "none",
     version: manifest.version.raw,
     oldVersion: publishedVersion.raw,
+    dryRun: options.dryRun
   };
 
   options.debug("OUTPUT:", results);

src/results.ts

@@ -3,13 +3,13 @@ import { ReleaseType } from "semver";
 export { ReleaseType };
 
 /**
- * Results of the `publish
+ * Results of the publish
  */
 export interface Results {
   /**
    * The type of version change that occurred
    */
-  type: ReleaseType | "none" | "dry-run";
+  type: ReleaseType | "none";
 
   /**
    * The name of the NPM package that was published
@@ -25,4 +25,9 @@ export interface Results {
    * The version number that was previously published to NPM
    */
   oldVersion: string;
+
+  /**
+   * Whether this was a dry run (not published to NPM)
+   */
+  dryRun: boolean;
 }