Skip to content

Commit

Permalink
change writeHash to use a default parameter instead of bind (thdk#17)
Browse files Browse the repository at this point in the history
  • Loading branch information
kinland committed Oct 19, 2023
1 parent 7a33bba commit 7625df8
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions src/is-package-changed.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { getPackagelock } from './get-packagelock';
interface PackageChangedResult {
hash: string;
oldHash: string | undefined;
writeHash(): void;
writeHash(hash?: string): void;
isChanged: boolean;
}

Expand Down Expand Up @@ -45,8 +45,6 @@ async function isPackageChanged(
}

const packageHashPath = path.join(cwd, hashFilename);
const writeHash = (hash: string | undefined) =>
hash && fs.writeFileSync(packageHashPath, hash, {});

const packageHashPathExists = fs.existsSync(packageHashPath);
const recentDigest = lockfile
Expand All @@ -64,6 +62,9 @@ async function isPackageChanged(
oldHash: previousDigest || undefined,
};

const writeHash = (hash: string | undefined = result.hash) =>
hash && fs.writeFileSync(packageHashPath, hash, {});

if (callback) {
let canWriteHash = await callback(result);
if (lockfile) {
Expand All @@ -82,7 +83,7 @@ async function isPackageChanged(

return {
...result,
writeHash: writeHash.bind(null, result.hash),
writeHash,
};
}

Expand Down

0 comments on commit 7625df8

Please sign in to comment.