Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

Commit

Permalink
Set num_threads to 1 now that rome uses a dedicated thread in the CLI…
Browse files Browse the repository at this point in the history
… and use powershell
  • Loading branch information
Micha Reiser committed Nov 7, 2022
1 parent ae86d1b commit f62d081
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions benchmark/run.js
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ function benchmarkFormatter(rome) {

const romeSingleCoreCommand = withEnvVariable(
"RAYON_NUM_THREADS",
"3",
"1",
romeCommand,
);

Expand Down Expand Up @@ -134,7 +134,7 @@ function benchmarkLinter(rome) {

const romeSingleCoreCommand = withEnvVariable(
"RAYON_NUM_THREADS",
"3",
"1",
romeCommand,
);

Expand All @@ -157,7 +157,7 @@ function shellOption() {
switch (process.platform) {
case "win32":
// Use Powershell so that it is possible to set an environment variable for a single command (ugh!)
return "pwsh";
return "powershell";
default:
return "default";
}
Expand Down

0 comments on commit f62d081

Please sign in to comment.