Skip to content

Use shared workflows for change detection, prek, and gate job#2104

Merged
KRRT7 merged 2 commits intomainfrom
ci/use-shared-workflows
Apr 23, 2026
Merged

Use shared workflows for change detection, prek, and gate job#2104
KRRT7 merged 2 commits intomainfrom
ci/use-shared-workflows

Conversation

@KRRT7
Copy link
Copy Markdown
Collaborator

@KRRT7 KRRT7 commented Apr 23, 2026

Relates to #2044

Summary

  • determine-changes: Replace inline shell-based change detection (merge-base + check_paths()) with codeflash-ai/github-workflows/.github/workflows/determine-changes.yml@main reusable workflow. All downstream jobs now read flags via fromJSON(needs.determine-changes.outputs.flags).FLAG_NAME instead of direct output references.
  • prek: Replace inline ruff auto-fix + prek-action steps with codeflash-ai/github-workflows/.github/workflows/prek-lint.yml@main reusable workflow, passing auto-fix, checkout-ref, and restore-paths inputs.
  • required-checks-passed: Replace inline jq verification script with codeflash-ai/github-workflows/.github/actions/required-checks-gate@main composite action.

All other jobs (check-linked-issue, unit-tests, coverage, type-check, e2e-python, e2e-js, e2e-java) are unchanged beyond the if: condition updates for the new output format.

Test plan

  • CI runs on this PR itself (the gate job, change detection, and prek all exercise the new shared workflows)
  • Verify determine-changes correctly detects this PR touches .github/workflows/ci.yaml and sets appropriate flags
  • Verify prek job runs and applies auto-fix if needed
  • Verify required-checks-passed gate job correctly aggregates all upstream results
  • Verify skipped jobs (e.g. e2e-java if no java paths changed) are treated as passing by the gate

Replace inline determine-changes, prek, and required-checks-passed
jobs with reusable workflows and composite actions from
codeflash-ai/github-workflows. This reduces CI maintenance burden
by centralizing common CI logic.
@github-actions github-actions Bot added the workflow-modified This PR modifies GitHub Actions workflows label Apr 23, 2026
@KRRT7 KRRT7 closed this Apr 23, 2026
@KRRT7 KRRT7 reopened this Apr 23, 2026
@KRRT7 KRRT7 merged commit cfdb131 into main Apr 23, 2026
14 checks passed
@KRRT7 KRRT7 deleted the ci/use-shared-workflows branch April 23, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflow-modified This PR modifies GitHub Actions workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant