Skip to content

fix(ci): avoid duplicate example builds in test job#1316

Merged
chaliy merged 1 commit intomainfrom
fix/main-ci-test-targets
Apr 15, 2026
Merged

fix(ci): avoid duplicate example builds in test job#1316
chaliy merged 1 commit intomainfrom
fix/main-ci-test-targets

Conversation

@chaliy
Copy link
Copy Markdown
Contributor

@chaliy chaliy commented Apr 15, 2026

Summary

  • stop the CI test job from rebuilding examples during cargo test
  • keep doctest coverage by running doc tests in a dedicated step
  • leave example coverage in the existing Examples job

Why

main went red on GitHub Actions run 24465694329 for commit e51470a because the Test job exhausted runner disk while linking python_scripts during cargo test --features http_client,ssh.

That job was compiling examples even though examples already have their own dedicated workflow job. Splitting the coverage avoids duplicate example links in the Test job and removes the disk-pressure path that broke main.

Test Plan

  • cargo test --workspace --lib --bins --tests --features http_client,ssh --no-run
  • cargo test --workspace --doc --features http_client,ssh
  • just pre-pr (fails on a pre-existing bash_comparison_tests mismatch in cargo test; unrelated to this workflow-only change)

@chaliy chaliy merged commit 4faad7e into main Apr 15, 2026
11 checks passed
@chaliy chaliy deleted the fix/main-ci-test-targets branch April 15, 2026 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant