Fix typo in hashing parsed Cargo.lock (#159)

This simple mistake caused the entire `Cargo.lock` to be ignored (JS treats having no return as `false`). Signed-off-by: Jiahao XU <Jiahao_XU@outlook.com>
Swatinem · Aug 10, 2023 · 4e0f4b1 · 4e0f4b1
1 parent b919e14
commit 4e0f4b1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/dist/restore/index.js b/dist/restore/index.js
@@ -67048,9 +67048,7 @@ class CacheConfig {
                     }
                     // Package without `[[package]].source` and `[[package]].checksum`
                     // are the one with `path = "..."` to crates within the workspace.
-                    const packages = parsed.package.filter((p) => {
-                        "source" in p || "checksum" in p;
-                    });
+                    const packages = parsed.package.filter((p) => "source" in p || "checksum" in p);
                     hasher.update(JSON.stringify(packages));
                     parsedKeyFiles.push(cargo_lock);
                 }

diff --git a/dist/save/index.js b/dist/save/index.js
@@ -67048,9 +67048,7 @@ class CacheConfig {
                     }
                     // Package without `[[package]].source` and `[[package]].checksum`
                     // are the one with `path = "..."` to crates within the workspace.
-                    const packages = parsed.package.filter((p) => {
-                        "source" in p || "checksum" in p;
-                    });
+                    const packages = parsed.package.filter((p) => "source" in p || "checksum" in p);
                     hasher.update(JSON.stringify(packages));
                     parsedKeyFiles.push(cargo_lock);
                 }

diff --git a/src/config.ts b/src/config.ts
@@ -198,9 +198,7 @@ export class CacheConfig {
 
           // Package without `[[package]].source` and `[[package]].checksum`
           // are the one with `path = "..."` to crates within the workspace.
-          const packages = parsed.package.filter((p: any) => {
-            "source" in p || "checksum" in p;
-          });
+          const packages = parsed.package.filter((p: any) => "source" in p || "checksum" in p);
 
           hasher.update(JSON.stringify(packages));