Update error message for no dependencies to cache (actions#968)

aparnajyothi-y · web-flow · commit 55aad42e4674 · 2024-11-04T19:27:40.000-06:00
diff --git a/dist/cache-save/index.js b/dist/cache-save/index.js
@@ -81165,7 +81165,7 @@ function saveCache(packageManager) {
         const cachePaths = JSON.parse(cachePathState);
         core.debug(`paths for caching are ${cachePaths.join(', ')}`);
         if (!isCacheDirectoryExists(cachePaths)) {
-            throw new Error(`Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePaths.join(', ')}`);
+            throw new Error(`Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePaths.join(', ')}. This likely indicates that there are no dependencies to cache. Consider removing the cache step if it is not needed.`);
         }
         const primaryKey = core.getState(cache_distributor_1.State.STATE_CACHE_PRIMARY_KEY);
         const matchedKey = core.getState(cache_distributor_1.State.CACHE_MATCHED_KEY);
diff --git a/src/cache-save.ts b/src/cache-save.ts
@@ -41,7 +41,7 @@ async function saveCache(packageManager: string) {
     throw new Error(
       `Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePaths.join(
         ', '
-      )}`
+      )}. This likely indicates that there are no dependencies to cache. Consider removing the cache step if it is not needed.`
     );
   }
 

Original file line number	Diff line number	Diff line change
`@@ -81165,7 +81165,7 @@ function saveCache(packageManager) {`
`81165`	`81165`	`const cachePaths = JSON.parse(cachePathState);`
`81166`	`81166`	core.debug(`paths for caching are ${cachePaths.join(', ')}`);
`81167`	`81167`	`if (!isCacheDirectoryExists(cachePaths)) {`
`81168`		- throw new Error(`Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePaths.join(', ')}`);
	`81168`	+ throw new Error(`Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePaths.join(', ')}. This likely indicates that there are no dependencies to cache. Consider removing the cache step if it is not needed.`);
`81169`	`81169`	`}`
`81170`	`81170`	`const primaryKey = core.getState(cache_distributor_1.State.STATE_CACHE_PRIMARY_KEY);`
`81171`	`81171`	`const matchedKey = core.getState(cache_distributor_1.State.CACHE_MATCHED_KEY);`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ async function saveCache(packageManager: string) {`
`41`	`41`	`throw new Error(`
`42`	`42`	`Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePaths.join(
`43`	`43`	`', '`
`44`		- )}`
	`44`	+ )}. This likely indicates that there are no dependencies to cache. Consider removing the cache step if it is not needed.`
`45`	`45`	`);`
`46`	`46`	`}`
`47`	`47`