Skip to content

Commit

Permalink
[ci] Remove separate npm install step. NFC (#23313)
Browse files Browse the repository at this point in the history
This is coverted by `./bootstrap` these days.
  • Loading branch information
sbc100 authored Jan 6, 2025
1 parent a927b40 commit c75230c
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,10 +55,6 @@ commands:
description: "bootstrap"
steps:
- run: ./bootstrap
npm-install:
description: "npm ci"
steps:
- run: npm ci
pip-install:
description: "pip install"
parameters:
Expand Down Expand Up @@ -164,7 +160,6 @@ commands:
cat ~/emsdk/.emscripten
- emsdk-env
- bootstrap
- npm-install
build-libs:
description: "Build all libraries"
steps:
Expand Down Expand Up @@ -223,7 +218,6 @@ commands:
command: git submodule update --init
- emsdk-env
- bootstrap
- npm-install
- pip-install
upload-test-results:
description: "Upload test results"
Expand Down Expand Up @@ -443,7 +437,7 @@ jobs:
executor: linux-node
steps:
- checkout
- npm-install
- run: npm ci
- run: |
npm run lint
npm run check
Expand Down

0 comments on commit c75230c

Please sign in to comment.