ci(typescript): gate SDK Devnet Test on sdk/contract changes - #929
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the TypeScript CI workflow to avoid running the SDK devnet E2E suite (which builds Cairo contracts + runs devnet tests) on PRs that don’t touch the SDK or Cairo/contracts-related files, while still keeping the required CI check reporting success.
Changes:
- Add a
dorny/paths-filtergate to theSDK Devnet Testjob based on changes tosdk/**,packages/**, and Scarb files. - Condition the devnet job’s expensive setup/build/test steps on the filter output so client-only PRs no-op the job.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware reviewed all commit messages and made 1 comment.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on ittaysw).
afdcdab to
4642457
Compare
Skip the Cairo build + devnet e2e on PRs that touch neither the SDK nor the contracts (e.g. client-only PRs), mirroring the existing paths-filter on the lint job. The steps no-op and the job still reports success, so a required check stays satisfied without gating client PRs on the unrelated devnet build.
4642457 to
4c10e4b
Compare
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on ittaysw).

Skip the Cairo build + devnet e2e on PRs that touch neither the SDK nor the
contracts (e.g. client-only PRs), mirroring the existing paths-filter on the
lint job. The steps no-op and the job still reports success, so a required
check stays satisfied without gating client PRs on the unrelated devnet build.
This change is