-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Open
Labels
feature requestNew feature or request to improve the current logicNew feature or request to improve the current logic
Description
Hi there,
at the moment, caching only hits if the exact same lockfile exists, but most of the time, even it the lockfile slightly changes, it is still beneficial to restore cache instead of downloading everything from scratch (most likely only ~1 dependency changed).
I would propose that, by default, cache falls back to node-cache-${platform}-${packageManager}
instead of only checking node-cache-${platform}-${packageManager}-${fileHash}
.
If this is of interest, I'm happy to provide a PR. It seems to only need a test and a one line change in https://github.com/actions/setup-node/blob/main/src/cache-restore.ts#L44
See also #304 (comment)
connorM43, Xetera, jdelStrother, ngraef, lharress and 30 more
Metadata
Metadata
Assignees
Labels
feature requestNew feature or request to improve the current logicNew feature or request to improve the current logic