Skip to content

Commit

Permalink
Merge pull request #54 from s-weigand/fix-53
Browse files Browse the repository at this point in the history
Fix Action fails if saving cache fails #53
  • Loading branch information
asottile committed Apr 5, 2021
2 parents 0404b96 + d1027a9 commit 8ac8946
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion index.js
Expand Up @@ -54,7 +54,20 @@ async function main() {
const restored = await cache.restoreCache(cachePaths, cacheKey);
const ret = await exec.exec('pre-commit', args, {ignoreReturnCode: push});
if (!restored) {
await cache.saveCache(cachePaths, cacheKey);
try {
await cache.saveCache(cachePaths, cacheKey);
} catch (e) {
core.warning(
`There was an error saving the pre-commit environments to cache:
${e.message || e}
This only has performance implications and won't change the result of your pre-commit tests.
If this problem persists on your default branch, you can try to fix it by editing your '.pre-commit-config.yaml'.
For example try to run 'pre-commit autoupdate' or simply add a blank line.
This will result in a different hash value and thus a different cache target.`.replace(/^ +/gm, '')
);
}
}

if (ret && push) {
Expand Down

0 comments on commit 8ac8946

Please sign in to comment.