mirror of
				https://gitea.com/actions/setup-python.git
				synced 2025-10-31 19:58:55 +08:00 
			
		
		
		
	Change missing cache directory error to warning (#1182)
* update error to warning when no dependency to cache * updated the return
This commit is contained in:
		
							
								
								
									
										3
									
								
								dist/cache-save/index.js
									
									
									
									
										vendored
									
									
								
							
							
						
						
									
										3
									
								
								dist/cache-save/index.js
									
									
									
									
										vendored
									
									
								
							| @@ -87883,7 +87883,8 @@ function saveCache(packageManager) { | |||||||
|         const cachePaths = JSON.parse(cachePathState); |         const cachePaths = JSON.parse(cachePathState); | ||||||
|         core.debug(`paths for caching are ${cachePaths.join(', ')}`); |         core.debug(`paths for caching are ${cachePaths.join(', ')}`); | ||||||
|         if (!isCacheDirectoryExists(cachePaths)) { |         if (!isCacheDirectoryExists(cachePaths)) { | ||||||
|             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.`); |             core.warning(`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.`); | ||||||
|  |             return; | ||||||
|         } |         } | ||||||
|         const primaryKey = core.getState(cache_distributor_1.State.STATE_CACHE_PRIMARY_KEY); |         const primaryKey = core.getState(cache_distributor_1.State.STATE_CACHE_PRIMARY_KEY); | ||||||
|         const matchedKey = core.getState(cache_distributor_1.State.CACHE_MATCHED_KEY); |         const matchedKey = core.getState(cache_distributor_1.State.CACHE_MATCHED_KEY); | ||||||
|   | |||||||
| @@ -38,11 +38,12 @@ async function saveCache(packageManager: string) { | |||||||
|   core.debug(`paths for caching are ${cachePaths.join(', ')}`); |   core.debug(`paths for caching are ${cachePaths.join(', ')}`); | ||||||
|  |  | ||||||
|   if (!isCacheDirectoryExists(cachePaths)) { |   if (!isCacheDirectoryExists(cachePaths)) { | ||||||
|     throw new Error( |     core.warning( | ||||||
|       `Cache folder path is retrieved for ${packageManager} but doesn't exist on disk: ${cachePaths.join( |       `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.` |       )}. This likely indicates that there are no dependencies to cache. Consider removing the cache step if it is not needed.` | ||||||
|     ); |     ); | ||||||
|  |     return; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   const primaryKey = core.getState(State.STATE_CACHE_PRIMARY_KEY); |   const primaryKey = core.getState(State.STATE_CACHE_PRIMARY_KEY); | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 aparnajyothi-y
					aparnajyothi-y