Removed check for unset UV_CACHE_DIR per PR comment

This commit is contained in:
James Braza 2025-10-01 12:48:52 -07:00
parent c8190078fd
commit 832b4cc483

View file

@ -55,16 +55,13 @@ async function saveCache(): Promise<void> {
// Check if UV_CACHE_DIR has been changed externally from this action
let actualCachePath = cacheLocalPath;
if (process.env.UV_CACHE_DIR !== cacheLocalPath) {
if (process.env.UV_CACHE_DIR) {
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}".`
);
actualCachePath = process.env.UV_CACHE_DIR;
} else {
core.warning(
`The environment variable UV_CACHE_DIR has been unset, by an action or step running after astral-sh/setup-uv. This can lead to unexpected behavior. If you expected this to happen unset cache-local-path input from "${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}".`,
);
actualCachePath = process.env.UV_CACHE_DIR!;
}
core.info(`Saving cache path: ${actualCachePath}`);