Skip to content

Commit

Permalink
chore(ci): only run doc tests on linux (#17379)
Browse files Browse the repository at this point in the history
Doc tests were observed to take over 100s on the Mac CI in one instance.
  • Loading branch information
dsherret committed Jan 13, 2023
1 parent 932d0b2 commit 94ec1a2
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 10 deletions.
13 changes: 8 additions & 5 deletions .github/workflows/ci.generate.ts
Original file line number Diff line number Diff line change
Expand Up @@ -592,14 +592,17 @@ const ci = {
"matrix.job == 'test' && matrix.profile == 'debug' &&",
"!startsWith(github.ref, 'refs/tags/')",
].join("\n"),
run: ["cargo test --locked --doc", "cargo test --locked"].join(
"\n",
),
run: "cargo test --locked",
},
{
name: "Test fastci",
if: "(matrix.job == 'test' && matrix.profile == 'fastci')",
run: "cargo test --locked",
if: "matrix.job == 'test' && matrix.profile == 'fastci'",
run: [
// Run unit then integration tests. Skip doc tests here
// since they are sometimes very slow on Mac.
"cargo test --locked --lib",
"cargo test --locked --test '*'",
].join("\n"),
env: {
CARGO_PROFILE_DEV_DEBUG: 0,
},
Expand Down
10 changes: 5 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -370,12 +370,12 @@ jobs:
if: |-
!(github.event_name == 'pull_request' && matrix.skip_pr) && (steps.exit_early.outputs.EXIT_EARLY != 'true' && (matrix.job == 'test' && matrix.profile == 'debug' &&
!startsWith(github.ref, 'refs/tags/')))
run: |-
cargo test --locked --doc
cargo test --locked
- name: Test fastci
if: '!(github.event_name == ''pull_request'' && matrix.skip_pr) && (steps.exit_early.outputs.EXIT_EARLY != ''true'' && ((matrix.job == ''test'' && matrix.profile == ''fastci'')))'
run: cargo test --locked
- name: Test fastci
if: '!(github.event_name == ''pull_request'' && matrix.skip_pr) && (steps.exit_early.outputs.EXIT_EARLY != ''true'' && (matrix.job == ''test'' && matrix.profile == ''fastci''))'
run: |-
cargo test --locked --lib
cargo test --locked --test '*'
env:
CARGO_PROFILE_DEV_DEBUG: 0
- name: Test release
Expand Down

0 comments on commit 94ec1a2

Please sign in to comment.