Skip to content
This repository was archived by the owner on Jul 1, 2024. It is now read-only.

Commit 78e3d76

Browse files
committed
Show differences from the required form
1 parent 96873ca commit 78e3d76

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

src/pr-info.ts

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -414,11 +414,18 @@ function makeJsonCheckerFromCore(requiredForm: any, ignoredKeys: string[]) {
414414
const newDiff = diffFromReq(contents);
415415
if (typeof newDiff === "string") return newDiff;
416416
if (newDiff.length === 0) return undefined;
417-
if (!oldText) return "not the required form";
417+
if (!oldText) return `not the required form
418+
\`\`\`JSON
419+
${JSON.stringify(newDiff)}
420+
\`\`\``;
418421
const oldDiff = diffFromReq(oldText);
419422
if (typeof oldDiff === "string") return oldDiff;
420-
if (jsonDiff.compare(oldDiff, newDiff).every(({ op }) => op === "remove")) return undefined;
421-
return "not the required form and not moving towards it";
423+
const notRemove = jsonDiff.compare(oldDiff, newDiff).filter(({ op }) => op !== "remove");
424+
if (notRemove.length === 0) return undefined;
425+
return `not the required form and not moving towards it
426+
\`\`\`JSON
427+
${JSON.stringify(notRemove.map(({ path }) => newDiff[Number(path.slice(1))]))}
428+
\`\`\``;
422429
};
423430
}
424431

0 commit comments

Comments
 (0)