Skip to content

Commit

Permalink
Propagate rustdoc links check to GH actions (#1446)
Browse files Browse the repository at this point in the history
* Propagate rustdoc links check to GH actions

* Fix WASM test

* Try fix WASM test again

* Try run tests on a real machine
  • Loading branch information
tomaka authored Feb 11, 2020
1 parent cbb3315 commit 627e93b
Showing 1 changed file with 28 additions and 5 deletions.
33 changes: 28 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,6 @@ jobs:
test-desktop:
name: Build and test
runs-on: ubuntu-latest
container:
image: rust
steps:
- uses: actions/checkout@v1
- name: Cache cargo registry
Expand All @@ -35,10 +33,12 @@ jobs:
run: cargo test --workspace --all-features

test-wasm:
name: Build and test
name: Build on WASM
runs-on: ubuntu-latest
container:
image: rust
env:
CC: clang-9
steps:
- uses: actions/checkout@v1
- name: Install Rust
Expand All @@ -47,6 +47,14 @@ jobs:
toolchain: stable
target: wasm32-unknown-unknown
override: true
- name: Install a recent version of clang
run: |
wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key | apt-key add -
echo "deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-9 main" >> /etc/apt/sources.list
apt-get update
apt-get install -y clang-9
- name: Install CMake
run: apt-get install -y cmake
- name: Cache cargo registry
uses: actions/cache@v1
with:
Expand All @@ -63,5 +71,20 @@ jobs:
path: target
key: wasm-cargo-build-target-${{ hashFiles('Cargo.toml') }}
- name: Build on WASM
# TODO: also run with `cargo test`
run: cargo build --workspace --target=wasm32-unknown-unknown
# TODO: also run `cargo test`
# TODO: ideally we would build `--workspace`, but not all crates compile for WASM
run: cargo build --target=wasm32-unknown-unknown

check-rustdoc-links:
name: Check rustdoc intra-doc links
runs-on: ubuntu-latest
container:
image: rust
steps:
- uses: actions/checkout@v1
- name: Install nightly Rust
# TODO: intra-doc links are available on nightly only
# see https://doc.rust-lang.org/nightly/rustdoc/lints.html#intra_doc_link_resolution_failure
run: rustup default nightly
- name: Check rustdoc links
run: RUSTDOCFLAGS="--deny intra_doc_link_resolution_failure" cargo +nightly doc --verbose --workspace --no-deps --document-private-items

0 comments on commit 627e93b

Please sign in to comment.