Skip to content

Add commcare_core_status_sha input for cross-repo CI#1766

Merged
dannyroberts merged 1 commit intomasterfrom
dmr/cross-repo-ci-merge-sha
Mar 18, 2026
Merged

Add commcare_core_status_sha input for cross-repo CI#1766
dannyroberts merged 1 commit intomasterfrom
dmr/cross-repo-ci-merge-sha

Conversation

@dannyroberts
Copy link
Member

@dannyroberts dannyroberts commented Mar 18, 2026

Technical Summary

Adds a new commcare_core_status_sha input to the cross-repo CI workflow dispatch.
This allows the commcare-core trigger workflow to send the merge commit SHA
(commcare_core_sha) for checkout — testing the same code commcare-core's own
build tested — while sending the PR head SHA (commcare_core_status_sha) for
reporting status back to the correct commit on the PR.

The status reporting step uses commcare_core_status_sha when provided, falling
back to commcare_core_sha for backward compatibility.

Cross-request: dimagi/commcare-core#1525

Safety Assurance

Safety story

Additive change — adds one optional input and adjusts which SHA is used for
status reporting. Existing behavior is unchanged when the new input is not provided.

Automated test coverage

CI infrastructure — validated by end-to-end testing.

QA Plan

Verified via cross-repo CI test runs.

Special deploy instructions

  • This PR can be deployed after merge with no further considerations.

Rollback instructions

  • This PR can be reverted after deploy with no further considerations.

Review

  • The set of people pinged as reviewers is appropriate for the level of risk of the change.

Allows the trigger workflow to send the merge commit SHA for checkout
while reporting status on the PR head SHA separately.
@codecov
Copy link

codecov bot commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.21%. Comparing base (3ee2a65) to head (ec185e5).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1766   +/-   ##
=========================================
  Coverage     70.21%   70.21%           
  Complexity     2029     2029           
=========================================
  Files           257      257           
  Lines          7996     7996           
  Branches        761      761           
=========================================
  Hits           5614     5614           
  Misses         2099     2099           
  Partials        283      283           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dannyroberts dannyroberts requested a review from shubham1g5 March 18, 2026 15:29
@dannyroberts dannyroberts marked this pull request as ready for review March 18, 2026 15:29
@dannyroberts dannyroberts merged commit a986d18 into master Mar 18, 2026
7 of 10 checks passed
@dannyroberts dannyroberts deleted the dmr/cross-repo-ci-merge-sha branch March 18, 2026 16:12
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