Skip to content

Commit

Permalink
build: print benchmark results at the end of yarn benchmarks run in…
Browse files Browse the repository at this point in the history
…vocation (angular#50816)

We already collect and report benchmark results for comparison runs, but
for normal benchmark runs we should do the same to make the results
more discoverable (and not in the middle of Bazel output).

PR Close angular#50816
  • Loading branch information
devversion authored and AndrewKushnir committed Jun 23, 2023
1 parent 232a786 commit bdd8813
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion scripts/benchmarks/index.mts
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,16 @@ async function runBenchmarkCmd(bazelTargetRaw: string | undefined): Promise<void
if (bazelTargetRaw === undefined) {
bazelTargetRaw = await promptForBenchmarkTarget();
}
await runBenchmarkTarget(await resolveTarget(bazelTargetRaw));
const bazelTarget = await resolveTarget(bazelTargetRaw);
const testlogPath = await getTestlogPath(bazelTarget);

await runBenchmarkTarget(bazelTarget);

const workingDirResults = await collectBenchmarkResults(testlogPath);

Log.info('\n\n\n');
Log.info(bold(green('Results!')));
Log.info(workingDirResults.summaryConsoleText);
}

/** Runs a benchmark Bazel target. */
Expand Down

0 comments on commit bdd8813

Please sign in to comment.