From c573adb13c5464b32caa6f0331bc8507146cd17f Mon Sep 17 00:00:00 2001 From: James Braza Date: Thu, 2 Oct 2025 08:05:32 -0700 Subject: [PATCH] Removed explanatory comment on assumption of UV_CACHE_DIR being set --- src/save-cache.ts | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/save-cache.ts b/src/save-cache.ts index a89815d..a0941f7 100644 --- a/src/save-cache.ts +++ b/src/save-cache.ts @@ -55,9 +55,6 @@ async function saveCache(): Promise { // Check if UV_CACHE_DIR has been changed externally from this action let actualCachePath = cacheLocalPath; if (process.env.UV_CACHE_DIR !== cacheLocalPath) { - // Don't need to check for unset UV_CACHE_DIR because: - // 1. We set the UV_CACHE_DIR earlier on in this action - // 2. Once an environment variable is set in GitHub Actions, it cannot be unset. core.warning( `The environment variable UV_CACHE_DIR has been changed to "${process.env.UV_CACHE_DIR}", by an action or step running after astral-sh/setup-uv. This can lead to unexpected behavior. If you expected this to happen set the cache-local-path input to "${process.env.UV_CACHE_DIR}" instead of "${cacheLocalPath}".`, );