Skip to content

Commit

Permalink
CI
Browse files Browse the repository at this point in the history
  • Loading branch information
robertbastian committed Jan 20, 2022
1 parent c8d2160 commit 61e7077
Showing 1 changed file with 13 additions and 10 deletions.
23 changes: 13 additions & 10 deletions .github/workflows/build-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -231,6 +231,10 @@ jobs:

steps:
- uses: actions/checkout@v2
- name: Install Node.js v14.17.0
uses: actions/setup-node@v1
with:
node-version: 14.17.0
- name: Load nightly Rust toolchain for WASM.
run: |
rustup install nightly-2021-12-22
Expand Down Expand Up @@ -320,15 +324,16 @@ jobs:
with:
command: make
args: wasm-release
- name: Install Node.js v14.17.0
uses: actions/setup-node@v1
with:
node-version: 14.17.0
- name: Build
- name: Test
uses: actions-rs/[email protected]
with:
command: make
args: wasm-test-release
- name: Build Examples and test icu4x-key-extract
uses: actions-rs/[email protected]
with:
command: make
args: wasm-compare-worklog-keys
# This has to be a separate test since the emscripten sdk
# will otherwise interfere with other node-using tests
- name: Run emscripten test
Expand Down Expand Up @@ -453,7 +458,7 @@ jobs:
with:
token: ${{ secrets.GITHUB_TOKEN }}
# keep args in sync with `clippy-all` in .cargo/config.toml
args: --all-targets --all-features -- -D warnings
args: --all-targets --all-features -- -D warnings -Aclippy::unknown-clippy-lints -Aclippy::field-reassign-with-default

# Benchmarking & dashboards job

Expand Down Expand Up @@ -759,10 +764,8 @@ jobs:
rustup component add rust-src
rustup toolchain list
rustup toolchain install nightly-2021-12-22
cd ffi/diplomat/wasm
npm ci
echo "$HOME/work/icu4x/icu4x/ffi/diplomat/wasm/node_modules/wasm-opt/bin" >> $GITHUB_PATH
echo "$HOME/work/icu4x/icu4x/ffi/diplomat/wasm/node_modules/wabt/bin" >> $GITHUB_PATH
sudo npm install -g wasm-opt --unsafe-perm
sudo npm install -g wabt
- name: Setup output data directory
run: |
Expand Down

0 comments on commit 61e7077

Please sign in to comment.