Skip to content

Commit

Permalink
Revert "ci: build only lib and bins (#10401)" (#10415)
Browse files Browse the repository at this point in the history
  • Loading branch information
ry authored Apr 29, 2021
1 parent 4e434d4 commit b9a136c
Showing 1 changed file with 7 additions and 11 deletions.
18 changes: 7 additions & 11 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ jobs:
path: |
./target
key:
b-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ hashFiles('Cargo.lock') }}
a-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ hashFiles('Cargo.lock') }}

- name: Apply and update mtime cache
uses: ./.github/mtime_cache
Expand All @@ -213,16 +213,12 @@ jobs:
run: deno run --unstable --allow-write --allow-read --allow-run ./tools/lint.js

- name: Build release
if: (matrix.kind == 'test') && matrix.profile == 'release'
run: cargo build --release --locked --bin deno --bin test_server -vv

- name: Build bench release
if: (matrix.kind == 'bench') && matrix.profile == 'release'
run: cargo build --release --locked --all-targets
if: (matrix.kind == 'test' || matrix.kind == 'bench') && matrix.profile == 'release'
run: cargo build --release --locked --all-targets -vv

- name: Build debug
if: (matrix.kind == 'test') && matrix.profile == 'debug'
run: cargo build --locked --bin deno --bin test_server --tests
if: (matrix.kind == 'test' || matrix.kind == 'bench') && matrix.profile == 'debug'
run: cargo build --locked --all-targets

- name: Pre-release (linux)
if: |
Expand Down Expand Up @@ -280,13 +276,13 @@ jobs:
- name: Test release
if: matrix.kind == 'test' && matrix.profile == 'release'
run: cargo test --release --locked --bins --lib --tests
run: cargo test --release --locked --all-targets

- name: Test debug
if: matrix.kind == 'test' && matrix.profile == 'debug'
run: |
cargo test --locked --doc
cargo test --locked --bins --lib --tests
cargo test --locked --all-targets
# TODO(ry) Because CI is so slow on for OSX and Windows, we currently only run WPT on Linux.
- name: Configure hosts file for WPT (linux)
Expand Down

0 comments on commit b9a136c

Please sign in to comment.