Skip to content

Commit

Permalink
Remove usage of yarn from critical path.
Browse files Browse the repository at this point in the history
Due to [this](yarnpkg/yarn#2165) and similar
issues, yarn doesn't refresh the local cache, and prevents local
package updates.
  • Loading branch information
shachlanAmazon committed Nov 7, 2022
1 parent 5a97498 commit c64acc2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
10 changes: 5 additions & 5 deletions benchmarks/install_and_test.sh
Expand Up @@ -42,15 +42,15 @@ function runPythonBenchmark(){
}

function runNodeBenchmark(){
yarn install
npm install
cd ${BENCH_FOLDER}/../node
yarn install
npm install
rm -rf build-ts
yarn run build
npm run build
cd ${BENCH_FOLDER}/node
yarn install
npm install
npx tsc
yarn run bench --resultsFile=../$1 --dataSize $2 --concurrentTasks $concurrentTasks --clients $chosenClients --host $host
npm run bench -- --resultsFile=../$1 --dataSize $2 --concurrentTasks $concurrentTasks --clients $chosenClients --host $host
}

function runCSharpBenchmark(){
Expand Down
4 changes: 2 additions & 2 deletions node/package.json
Expand Up @@ -348,9 +348,9 @@
},
"scripts": {
"initial-build": "npm install --location=global yarn && cd rust-client && yarn install",
"build": "yarn build-internal && yarn build-external",
"build": "npm run build-internal && npm run build-external",
"build-internal": "cd rust-client && yarn build",
"build-external": "rm -rf build-ts && yarn tsc",
"build-external": "rm -rf build-ts && npx tsc",
"test": "jest"
},
"author": "",
Expand Down

0 comments on commit c64acc2

Please sign in to comment.