Skip to content

Commit

Permalink
call getSingleResultLogMessage() after each document, not each file
Browse files Browse the repository at this point in the history
  • Loading branch information
chris48s committed Aug 25, 2024
1 parent e44be46 commit 3b64d6b
Showing 1 changed file with 23 additions and 26 deletions.
49 changes: 23 additions & 26 deletions src/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -124,18 +124,30 @@ async function validateFile(filename, config, plugins, cache) {
let results = [];
for (let i = 0; i < documents.length; i++) {
const documentIndex = documents.length === 1 ? null : i;
results.push(
await validateDocument(
filename,
documentIndex,
documents[i],
schemaLocation,
schema,
strictMode,
cache,
resolver,
),
const result = await validateDocument(
filename,
documentIndex,
documents[i],
schemaLocation,
schema,
strictMode,
cache,
resolver,
);

results.push(result);

for (const plugin of plugins) {
const message = plugin.getSingleResultLogMessage(
result,
filename,
config.format,
);
if (message != null) {
logger.log(message);
break;
}
}
}
return results;
}
Expand Down Expand Up @@ -174,21 +186,6 @@ function Validator() {
for (const filename of filenames) {
const fileResults = await validateFile(filename, config, plugins, cache);
results = results.concat(fileResults);

for (const result of results) {
for (const plugin of plugins) {
const message = plugin.getSingleResultLogMessage(
result,
filename,
config.format,
);
if (message != null) {
logger.log(message);
break;
}
}
}

cache.resetCounters();
}

Expand Down

0 comments on commit 3b64d6b

Please sign in to comment.