Changing flag to be clearer

Author: Tohaker

src/action/index.ts

@@ -23,7 +23,7 @@ async function main(): Promise<void> {
       tag: getInput("tag"),
       access: getInput("access") as Access,
       dryRun: getInput("dry-run").toLowerCase() === "true",
-      greaterVersion: getInput("greater-version").toLowerCase() === "true",
+      greaterVersionOnly: getInput("greater-version-only").toLowerCase() === "true",
       debug: debugHandler,
     };
 

src/normalize-options.ts

@@ -13,7 +13,7 @@ export interface NormalizedOptions {
   access?: Access;
   dryRun: boolean;
   checkVersion: boolean;
-  greaterVersion: boolean;
+  greaterVersionOnly: boolean;
   quiet: boolean;
   debug: Debug;
 }
@@ -37,10 +37,10 @@ export function normalizeOptions(options: Options): NormalizedOptions {
     dryRun: options.dryRun || false,
     checkVersion:
       options.checkVersion === undefined ? true : Boolean(options.checkVersion),
-    greaterVersion:
-      options.greaterVersion === undefined
+    greaterVersionOnly:
+      options.greaterVersionOnly === undefined
         ? false
-        : Boolean(options.greaterVersion),
+        : Boolean(options.greaterVersionOnly),
     quiet: options.quiet || false,
     debug: options.debug || (() => undefined),
   };

src/npm-publish.ts

@@ -24,7 +24,7 @@ export async function npmPublish(opts: Options = {}): Promise<Results> {
   let shouldPublish =
     !options.checkVersion ||
     // compare returns 1 if manifest is higher than published
-    (options.greaterVersion && cmp === 1) ||
+    (options.greaterVersionOnly && cmp === 1) ||
     // compare returns 0 if the manifest is the same as published
     cmp !== 0;
 
@@ -33,13 +33,11 @@ export async function npmPublish(opts: Options = {}): Promise<Results> {
     await npm.publish(manifest, options);
   }
 
-  // The version should be marked as lower if we disallow decrementing the version
-  let type =
-    (options.greaterVersion && cmp === -1 && "lower") || diff || "none";
-
   let results: Results = {
     package: manifest.name,
-    type,
+    // The version should be marked as lower if we disallow decrementing the version
+    type:
+      (options.greaterVersionOnly && cmp === -1 && "lower") || diff || "none",
     version: manifest.version.raw,
     oldVersion: publishedVersion.raw,
     tag: options.tag,

src/options.ts

@@ -70,7 +70,7 @@ export interface Options {
    *
    * Defaults to `false`
    */
-  greaterVersion?: boolean;
+  greaterVersionOnly?: boolean;
 
   /**
    * A function to call to log debug messages.

test/specs/action/success.spec.js

@@ -155,7 +155,7 @@ describe("GitHub Action - success tests", () => {
     let cli = exec.action({
       env: {
         INPUT_TOKEN: "my-secret-token",
-        "INPUT_GREATER-VERSION": "true",
+        "INPUT_GREATER-VERSION-ONLY": "true",
       },
     });