From 34faf3ad7ca405c58e3ee0440790e28adedffa28 Mon Sep 17 00:00:00 2001
From: Sankalp Kotewar <98868223+kotewar@users.noreply.github.com>
Date: Wed, 21 Dec 2022 06:25:59 +0000
Subject: [PATCH] Updated return ID to -2

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

diff --git a/dist/save-only/index.js b/dist/save-only/index.js
index 3f87a6f..b0ae780 100644
--- a/dist/save-only/index.js
+++ b/dist/save-only/index.js
@@ -41122,11 +41122,11 @@ function saveImpl(stateProvider) {
         let cacheId;
         try {
             if (!utils.isCacheFeatureAvailable()) {
-                return 0;
+                return -2; //-2 refers as safe to ignore for the caller
             }
             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 0;
+                return -2;
             }
             // 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 0;
+                return -2;
             }
             // 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 0;
+                return -2;
             }
             const cachePaths = utils.getInputAsArray(constants_1.Inputs.Path, {
                 required: true
@@ -41149,6 +41149,7 @@ function saveImpl(stateProvider) {
             cacheId = yield cache.saveCache(cachePaths, primaryKey, {
                 uploadChunkSize: utils.getInputAsInt(constants_1.Inputs.UploadChunkSize)
             });
+            // -1 refers to cache not saved
             if (cacheId != -1) {
                 core.info(`Cache saved with key: ${primaryKey}`);
             }
diff --git a/dist/save/index.js b/dist/save/index.js
index 5870eb4..2bad0dd 100644
--- a/dist/save/index.js
+++ b/dist/save/index.js
@@ -41066,11 +41066,11 @@ function saveImpl(stateProvider) {
         let cacheId;
         try {
             if (!utils.isCacheFeatureAvailable()) {
-                return 0;
+                return -2; //-2 refers as safe to ignore for the caller
             }
             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 0;
+                return -2;
             }
             // 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 0;
+                return -2;
             }
             // 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 0;
+                return -2;
             }
             const cachePaths = utils.getInputAsArray(constants_1.Inputs.Path, {
                 required: true
@@ -41093,6 +41093,7 @@ function saveImpl(stateProvider) {
             cacheId = yield cache.saveCache(cachePaths, primaryKey, {
                 uploadChunkSize: utils.getInputAsInt(constants_1.Inputs.UploadChunkSize)
             });
+            // -1 refers to cache not saved
             if (cacheId != -1) {
                 core.info(`Cache saved with key: ${primaryKey}`);
             }
diff --git a/src/saveImpl.ts b/src/saveImpl.ts
index 9f86e52..9e9f983 100644
--- a/src/saveImpl.ts
+++ b/src/saveImpl.ts
@@ -14,7 +14,7 @@ async function saveImpl(stateProvider: IStateProvider): Promise<number> {
     let cacheId;
     try {
         if (!utils.isCacheFeatureAvailable()) {
-            return 0;
+            return -2; //-2 refers as safe to ignore for the caller
         }
 
         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 0;
+            return -2;
         }
 
         // 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 0;
+            return -2;
         }
 
         // 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 0;
+            return -2;
         }
 
         const cachePaths = utils.getInputAsArray(Inputs.Path, {
@@ -56,6 +56,7 @@ async function saveImpl(stateProvider: IStateProvider): Promise<number> {
             uploadChunkSize: utils.getInputAsInt(Inputs.UploadChunkSize)
         });
 
+        // -1 refers to cache not saved
         if (cacheId != -1) {
             core.info(`Cache saved with key: ${primaryKey}`);
         }