From f9ae5e0028e6c0f2d15a290a24def85946cef27b Mon Sep 17 00:00:00 2001
From: Sankalp Kotewar <98868223+kotewar@users.noreply.github.com>
Date: Wed, 21 Dec 2022 08:18:24 +0000
Subject: [PATCH] Removed -2 error code

---
 dist/save-only/index.js | 10 +++++-----
 dist/save/index.js      | 10 +++++-----
 src/saveImpl.ts         | 12 ++++++------
 3 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/dist/save-only/index.js b/dist/save-only/index.js
index b0ae780..338d315 100644
--- a/dist/save-only/index.js
+++ b/dist/save-only/index.js
@@ -41119,14 +41119,14 @@ const utils = __importStar(__webpack_require__(443));
 process.on("uncaughtException", e => utils.logWarning(e.message));
 function saveImpl(stateProvider) {
     return __awaiter(this, void 0, void 0, function* () {
-        let cacheId;
+        let cacheId = -1;
         try {
             if (!utils.isCacheFeatureAvailable()) {
-                return -2; //-2 refers as safe to ignore for the caller
+                return;
             }
             if (!utils.isValidEvent()) {
                 utils.logWarning(`Event Validation Error: The event type ${process.env[constants_1.Events.Key]} is not supported because it's not tied to a branch or tag ref.`);
-                return -2;
+                return;
             }
             // If restore has stored a primary key in state, reuse that
             // Else re-evaluate from inputs
@@ -41134,14 +41134,14 @@ function saveImpl(stateProvider) {
                 core.getInput(constants_1.Inputs.Key);
             if (!primaryKey) {
                 utils.logWarning(`Key is not specified.`);
-                return -2;
+                return;
             }
             // If matched restore key is same as primary key, then do not save cache
             // NO-OP in case of SaveOnly action
             const restoredKey = stateProvider.getCacheState();
             if (utils.isExactKeyMatch(primaryKey, restoredKey)) {
                 core.info(`Cache hit occurred on the primary key ${primaryKey}, not saving cache.`);
-                return -2;
+                return;
             }
             const cachePaths = utils.getInputAsArray(constants_1.Inputs.Path, {
                 required: true
diff --git a/dist/save/index.js b/dist/save/index.js
index 2bad0dd..e6269b4 100644
--- a/dist/save/index.js
+++ b/dist/save/index.js
@@ -41063,14 +41063,14 @@ const utils = __importStar(__webpack_require__(443));
 process.on("uncaughtException", e => utils.logWarning(e.message));
 function saveImpl(stateProvider) {
     return __awaiter(this, void 0, void 0, function* () {
-        let cacheId;
+        let cacheId = -1;
         try {
             if (!utils.isCacheFeatureAvailable()) {
-                return -2; //-2 refers as safe to ignore for the caller
+                return;
             }
             if (!utils.isValidEvent()) {
                 utils.logWarning(`Event Validation Error: The event type ${process.env[constants_1.Events.Key]} is not supported because it's not tied to a branch or tag ref.`);
-                return -2;
+                return;
             }
             // If restore has stored a primary key in state, reuse that
             // Else re-evaluate from inputs
@@ -41078,14 +41078,14 @@ function saveImpl(stateProvider) {
                 core.getInput(constants_1.Inputs.Key);
             if (!primaryKey) {
                 utils.logWarning(`Key is not specified.`);
-                return -2;
+                return;
             }
             // If matched restore key is same as primary key, then do not save cache
             // NO-OP in case of SaveOnly action
             const restoredKey = stateProvider.getCacheState();
             if (utils.isExactKeyMatch(primaryKey, restoredKey)) {
                 core.info(`Cache hit occurred on the primary key ${primaryKey}, not saving cache.`);
-                return -2;
+                return;
             }
             const cachePaths = utils.getInputAsArray(constants_1.Inputs.Path, {
                 required: true
diff --git a/src/saveImpl.ts b/src/saveImpl.ts
index 9e9f983..6c4e904 100644
--- a/src/saveImpl.ts
+++ b/src/saveImpl.ts
@@ -10,11 +10,11 @@ import * as utils from "./utils/actionUtils";
 // throw an uncaught exception.  Instead of failing this action, just warn.
 process.on("uncaughtException", e => utils.logWarning(e.message));
 
-async function saveImpl(stateProvider: IStateProvider): Promise<number> {
-    let cacheId;
+async function saveImpl(stateProvider: IStateProvider): Promise<number | void> {
+    let cacheId = -1;
     try {
         if (!utils.isCacheFeatureAvailable()) {
-            return -2; //-2 refers as safe to ignore for the caller
+            return;
         }
 
         if (!utils.isValidEvent()) {
@@ -23,7 +23,7 @@ async function saveImpl(stateProvider: IStateProvider): Promise<number> {
                     process.env[Events.Key]
                 } is not supported because it's not tied to a branch or tag ref.`
             );
-            return -2;
+            return;
         }
 
         // If restore has stored a primary key in state, reuse that
@@ -34,7 +34,7 @@ async function saveImpl(stateProvider: IStateProvider): Promise<number> {
 
         if (!primaryKey) {
             utils.logWarning(`Key is not specified.`);
-            return -2;
+            return;
         }
 
         // If matched restore key is same as primary key, then do not save cache
@@ -45,7 +45,7 @@ async function saveImpl(stateProvider: IStateProvider): Promise<number> {
             core.info(
                 `Cache hit occurred on the primary key ${primaryKey}, not saving cache.`
             );
-            return -2;
+            return;
         }
 
         const cachePaths = utils.getInputAsArray(Inputs.Path, {