Skip to content

feat(ci): advanced triggers rollout for unit and integration tests#22054

Merged
erikburt merged 2 commits intodevelopfrom
feat/ci-advanced-triggers
Apr 27, 2026
Merged

feat(ci): advanced triggers rollout for unit and integration tests#22054
erikburt merged 2 commits intodevelopfrom
feat/ci-advanced-triggers

Conversation

@erikburt
Copy link
Copy Markdown
Collaborator

This adds an advanced-triggers step which in the future will be used to gate jobs more effectively. The outputs of this are currently unused so we can validate the behaviour while still relying on the known-working existing conditional logic.

Utilizes advanced-triggers.

Changes

  • Adds advanced-triggers step to both ci-core and integration-tests

Motivation

This provides more granular control for job-level run conditions based on file changes.

  • Allows for more intuitive reusable glob groupings, and proper negations

Notes

This replaces #21686, #21685.


DX-3534

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 16, 2026

✅ No conflicts with other open PRs targeting develop

@erikburt erikburt marked this pull request as ready for review April 16, 2026 21:43
@erikburt erikburt requested review from a team as code owners April 16, 2026 21:43
Copilot AI review requested due to automatic review settings April 16, 2026 21:43
@erikburt erikburt requested a review from a team as a code owner April 16, 2026 21:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Risk Rating: LOW — adds non-gating CI steps (with continue-on-error) but still touches critical workflows and can impact rollout validation quality.

This PR introduces the smartcontractkit/.github/actions/advanced-triggers step into existing CI workflows to begin a rollout phase where trigger behavior can be observed without changing current job gating.

Changes:

  • Add an advanced-triggers step (rollout-only) to ci-core’s change-detection job.
  • Add an advanced-triggers step (rollout-only) to integration-tests’ path-checking job.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
.github/workflows/ci-core.yml Adds advanced-triggers trigger evaluation alongside existing dorny/paths-filter logic.
.github/workflows/integration-tests.yml Adds advanced-triggers trigger evaluation alongside existing dorny/paths-filter logic.

Comment thread .github/workflows/ci-core.yml Outdated
Comment thread .github/workflows/ci-core.yml Outdated
Comment thread .github/workflows/ci-core.yml
Comment thread .github/workflows/integration-tests.yml Outdated
Comment thread .github/workflows/integration-tests.yml
@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Apr 16, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@cl-sonarqube-production
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@erikburt erikburt requested a review from kalverra April 17, 2026 21:37
@erikburt erikburt added this pull request to the merge queue Apr 27, 2026
Merged via the queue into develop with commit 3ff1702 Apr 27, 2026
114 checks passed
@erikburt erikburt deleted the feat/ci-advanced-triggers branch April 27, 2026 23:01
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.

4 participants