actions/setup-node
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge dff445bec7 into 04be95cf35

Browse source
This commit is contained in:
Satishchoudhary94 2026-07-08 12:16:24 -04:00 • committed by GitHub
commit 7e15a936f3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
5 changed files with 109 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/cache-save.ts
View file

@ -2,7 +2,7 @@ import * as core from '@actions/core';
import * as cache from '@actions/cache';
import {State} from './constants';
import {getPackageManagerInfo} from './cache-utils';
import {getPackageManagerInfo, isPackageManagerInstalled} from './cache-utils';
// Catch and log any unhandled exceptions. These exceptions can leak out of the uploadChunk method in
// @actions/toolkit when a failed upload closes the file descriptor causing any in-process reads to
@ -45,6 +45,17 @@ const cachePackages = async (packageManager: string) => {
return;
}
// Check if the package manager is installed before attempting to save cache
// This prevents cache save failures for package managers that may not be installed
const isInstalled = await isPackageManagerInstalled(packageManager);
if (!isInstalled) {
core.warning(
`Package manager '${packageManager}' was not found in the PATH. ` +
`Skipping cache save.`
);
return;
}
if (!cachePaths.length) {
// TODO: core.getInput has a bug - it can return undefined despite its definition (tests only?)
// export declare function getInput(name: string, options?: InputOptions): string;