Skip to content

Commit 3e49948

Browse files
Merge pull request #1207 from github/edoardo/fix-warning
TRAP Caching: Unset missing cache rather than setting to `undefined`
2 parents c2c0a29 + 1e5376a commit 3e49948

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

lib/trap-caching.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/trap-caching.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/trap-caching.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ export async function downloadTrapCaches(
113113
// still just an empty directory. There's no reason to tell the extractor to use it,
114114
// so let's unset the entry in the map so we don't set any extractor options.
115115
logger.info(`No TRAP cache found in Actions cache for ${language}`);
116-
result[language] = undefined;
116+
delete result[language];
117117
}
118118
}
119119

0 commit comments

Comments
 (0)