Skip to content

Commit

Permalink
refactor: no need for a workaround for actions/toolkit#1378 (comment)
Browse files Browse the repository at this point in the history
…since we use the latest @cache-nix-action/cache which is a mod of @actions/cache which has this bug fixed actions/cache#1302
  • Loading branch information
deemp committed Mar 10, 2024
1 parent 245bf2e commit d6e2920
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
3 changes: 1 addition & 2 deletions src/utils/action.ts
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,7 @@ export async function restoreCache({
lookupOnly: boolean;
}) {
return await cache.restoreCache(
// https://github.com/actions/toolkit/pull/1378#issuecomment-1478388929
inputs.paths.slice(),
inputs.paths,
primaryKey,
restoreKeys,
{ lookupOnly },
Expand Down
4 changes: 1 addition & 3 deletions src/utils/purge.ts
Original file line number Diff line number Diff line change
Expand Up @@ -149,8 +149,6 @@ export async function purgeCachesByTime({
time: number;
prefixes: string[];
}): Promise<void> {
// TODO https://github.com/actions/toolkit/pull/1378#issuecomment-1478388929

for (const doUseLastAccessedTime of [true, false]) {
if (
(doUseLastAccessedTime
Expand All @@ -160,7 +158,7 @@ export async function purgeCachesByTime({
await purgeByTime({
primaryKey,
doUseLastAccessedTime,
prefixes: prefixes.slice(),
prefixes,
time
});
}
Expand Down

0 comments on commit d6e2920

Please sign in to comment.