From 2b28ef3fe9965069e060010ff92f799c133ba478 Mon Sep 17 00:00:00 2001 From: Guillermo Perez Date: Fri, 24 Nov 2023 20:10:44 +0100 Subject: [PATCH] CI bug fixes - Part III (#1625) * Add docs check to the PR sanity checks * Disable the docs publishing for PRs --- .github/workflows/{docs.yml => deploy-docs.yml} | 1 - .github/workflows/sanity-checks.yml | 6 +++--- ci/run-check.sh | 3 +++ 3 files changed, 6 insertions(+), 4 deletions(-) rename .github/workflows/{docs.yml => deploy-docs.yml} (99%) diff --git a/.github/workflows/docs.yml b/.github/workflows/deploy-docs.yml similarity index 99% rename from .github/workflows/docs.yml rename to .github/workflows/deploy-docs.yml index f74cb6f8ad..82bc668e9e 100644 --- a/.github/workflows/docs.yml +++ b/.github/workflows/deploy-docs.yml @@ -2,7 +2,6 @@ name: Rustdoc on: push: branches: [main] - pull_request: concurrency: group: ${{ github.workflow }}-${{ github.event.pull_request.head.label || github.head_ref || github.ref }} cancel-in-progress: true diff --git a/.github/workflows/sanity-checks.yml b/.github/workflows/sanity-checks.yml index 60f4abe409..ea48608011 100644 --- a/.github/workflows/sanity-checks.yml +++ b/.github/workflows/sanity-checks.yml @@ -14,7 +14,7 @@ jobs: strategy: matrix: target: [test-general, test-integration, - lint-fmt, lint-clippy, cargo-build] # ,lint-taplo] + lint-fmt, lint-clippy, cargo-build, docs-build] # ,lint-taplo] steps: - name: Check out code uses: actions/checkout@8e5e7e5ab8b370d6c329ec480221332ada57f0ab #3.5.2 @@ -31,7 +31,7 @@ jobs: run: ./ci/run-check.sh env: TARGET: ${{ matrix.target }} - RUSTC_WRAPPER: "sccache" + RUSTC_WRAPPER: "sccache" benchmark-check: name: bench-check-${{ matrix.runtime }} @@ -55,4 +55,4 @@ jobs: env: TARGET: benchmark-check RUNTIME: ${{ matrix.runtime }} - RUSTC_WRAPPER: "sccache" + RUSTC_WRAPPER: "sccache" \ No newline at end of file diff --git a/ci/run-check.sh b/ci/run-check.sh index f3fe6b094b..b266224471 100755 --- a/ci/run-check.sh +++ b/ci/run-check.sh @@ -34,4 +34,7 @@ case $TARGET in ;; benchmark-check) ./scripts/check_benchmarks.sh $RUNTIME + ;; + docs-build) + RUSTDOCFLAGS="-D warnings" cargo doc --all --no-deps esac