Skip to content

W-23480663 ci: skip regression tests on PRs, run them only on master#127

Merged
mlischetti merged 1 commit into
masterfrom
ci/skip-regression-tests-on-prs
Jul 16, 2026
Merged

W-23480663 ci: skip regression tests on PRs, run them only on master#127
mlischetti merged 1 commit into
masterfrom
ci/skip-regression-tests-on-prs

Conversation

@mlischetti

Copy link
Copy Markdown
Contributor

Extracted from #119 as an independent CI cost improvement.

Change

The 2.9.8 and 2.10 regression suites (native-cli-integration-tests) run on every push and pull_request, adding significant time to PR builds. Gate both steps behind github.ref == 'refs/heads/master' so they run on master pushes but are skipped on PRs — the main build + unit tests still run on PRs.

Notes

  • Branches off master; +3/−1 lines to .github/workflows/main.yml, valid YAML.
  • Contains none of Native-bindings #119's new-binding CI plumbing (Setup Go/Rust/CMake, packageGo/Rust/C, bindings bundle) — those depend on tasks/dirs that don't exist on master.

The 2.9.8 and 2.10 regression suites (native-cli-integration-tests) run on
every push and pull_request, adding significant time to PR builds. Gate both
steps behind `github.ref == 'refs/heads/master'` so they run on master pushes
but are skipped on PRs, where the main build + unit tests still run.

Extracted from the native-bindings branch (feat/new-native-bindings) as an
independent CI cost improvement — none of the new-binding plumbing from that
branch is included.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mlischetti
mlischetti requested a review from a team as a code owner July 16, 2026 19:48
@mlischetti
mlischetti merged commit 4fc9c6c into master Jul 16, 2026
3 checks passed
@mlischetti
mlischetti deleted the ci/skip-regression-tests-on-prs branch July 16, 2026 20:03
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.

2 participants