Skip to content

ci: split inspect and sync lanes in depot ci#17

Merged
anandpant merged 2 commits into
mainfrom
feat/ci-lane-split-v2
Apr 20, 2026
Merged

ci: split inspect and sync lanes in depot ci#17
anandpant merged 2 commits into
mainfrom
feat/ci-lane-split-v2

Conversation

@anandpant

@anandpant anandpant commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a changed-path classifier job to Depot CI
  • split Rust verification into inspect/core lanes and sync/export lanes
  • avoid rebuilding the heavy S3/AWS/Arrow stack on clearly unrelated PRs

What Changed

  • add Changed Paths job
  • split CI into:
    • Clippy / Inspect
    • Test / Inspect
    • Clippy / Sync
    • Test / Sync
  • gate inspect lanes on:
    • apps/convex-inspect/**
    • crates/convex-sync-core/**
    • shared workspace files
  • gate sync lanes on:
    • apps/convex-sync/**
    • crates/convex-export-s3/**
    • shared workspace files
  • update the top-level README verification section to match the new CI shape

Why

The binary split is already merged. This PR is the operational payoff:

  • light/source-only work can finish sooner
  • heavy sync/export work remains isolated in its own lanes

Verification

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo test --workspace
  • actionlint -config-file .github/actionlint.yaml .github/workflows/*.yml
  • live Depot CI proof against this branch:
    • run mbt4ztm544
    • confirmed jobs fan out as:
      • Changed Paths
      • Clippy / Inspect
      • Test / Inspect
      • Clippy / Sync
      • Test / Sync

Open in Devin Review

devin-ai-integration[bot]

This comment was marked as resolved.

@anandpant
anandpant merged commit 33d2e9b into main Apr 20, 2026
9 checks passed
@anandpant
anandpant deleted the feat/ci-lane-split-v2 branch April 20, 2026 23:35
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