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

Commit efd2891

Browse files
committed
Show differences from the required form
1 parent a289014 commit efd2891

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
@@ -416,11 +416,18 @@ function makeJsonCheckerFromCore(requiredForm: any, ignoredKeys: string[]) {
416416
const newDiff = diffFromReq(contents);
417417
if (typeof newDiff === "string") return newDiff;
418418
if (newDiff.length === 0) return undefined;
419-
if (!oldText) return "not the required form";
419+
if (!oldText) return `not the required form
420+
\`\`\`JSON
421+
${JSON.stringify(newDiff)}
422+
\`\`\``;
420423
const oldDiff = diffFromReq(oldText);
421424
if (typeof oldDiff === "string") return oldDiff;
422-
if (jsonDiff.compare(oldDiff, newDiff).every(({ op }) => op === "remove")) return undefined;
423-
return "not the required form and not moving towards it";
425+
const notRemove = jsonDiff.compare(oldDiff, newDiff).filter(({ op }) => op !== "remove");
426+
if (notRemove.length === 0) return undefined;
427+
return `not the required form and not moving towards it
428+
\`\`\`JSON
429+
${JSON.stringify(notRemove.map(({ path }) => newDiff[Number(path.slice(1))]))}
430+
\`\`\``;
424431
};
425432
}
426433

0 commit comments

Comments
 (0)