Skip to content

ci: optimize workflow caching and setup#880

Merged
loonghao merged 5 commits into
mainfrom
fix/coauthor-email
May 20, 2026
Merged

ci: optimize workflow caching and setup#880
loonghao merged 5 commits into
mainfrom
fix/coauthor-email

Conversation

@loonghao
Copy link
Copy Markdown
Owner

@loonghao loonghao commented May 19, 2026

Summary

  • Avoid duplicate vx tool caching when workflows already manage ~/.vx explicitly.
  • Make the setup action cache key sensitive to vx.toml / vx.lock changes.
  • Reuse the shared Rust + sccache setup action in provider/action test workflows instead of repeating setup blocks.
  • Move expensive coverage generation to manual full-CI runs only.
  • Avoid direct github.head_ref interpolation in the workspace-hack push script.
  • Fix mirror-backed provider checks for ffmpeg and witr by using the tag-prefix-aware version fetcher.
  • Skip Miniforge conda / mamba runtimes in provider E2E because the installer layout is not extractable by the current provider test harness; micromamba remains covered.

Validation

  • vx actionlint .github/workflows/ci.yml .github/workflows/docs.yml .github/workflows/codeql.yml .github/workflows/security-audit.yml .github/workflows/test-action.yml .github/workflows/test-providers.yml
  • vx yq eval '.runs.using, .inputs.cache.default, .runs.steps[0].with.key' action.yml
  • vx git diff --check
  • vx cargo test -p vx-starlark --test lint_all_providers_test -- --nocapture
  • vx cargo build
  • target/debug/vx test --ci --ci-runtimes ffmpeg,witr --temp-root --keep-going --verbose
  • vx bash .github/scripts/discover-providers.sh --chunk-size 8

@loonghao loonghao changed the title fix: correct co-author email trailers ci: optimize workflow caching and setup May 19, 2026
@loonghao loonghao force-pushed the fix/coauthor-email branch from ba4cbe9 to ec9a76f Compare May 19, 2026 07:40
@loonghao loonghao marked this pull request as ready for review May 20, 2026 10:25
@loonghao loonghao merged commit 1a01df2 into main May 20, 2026
93 checks passed
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