Wrapped saving pre-commit cache to cache, in a try catch block

s-weigand · s-weigand · commit 470aac685352 · 2020-10-12T19:52:10.000+02:00
diff --git a/index.js b/index.js
@@ -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:\n\n",
+                    `${e.message}\n\n`,
+                    "This only has performance implications and won't change the result of your pre-commit tests.\n",
+                    "If this problem persists on your default branch, you can try to fix it by editing your '.pre-commit-config.yaml'. ",
+                    "I.e. run 'pre-commit autoupdate' or simply add a blank line. ",
+                    "This will result in a different hash value and thus a different cache target.",
+                ].join("")
+            );
+        }
     }
 
     if (ret && push) {
