From 2890d96d64de2213dbb68997c1eaa78b5a1c1203 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Miko=C5=82aj=20Poche=C4=87?= Date: Fri, 4 Sep 2026 20:49:13 +0200 Subject: [PATCH 1/2] Use pinned Mobile-Expensify commit in ad-hoc test builds --- .github/workflows/testBuild.yml | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/.github/workflows/testBuild.yml b/.github/workflows/testBuild.yml index 59fb802e7884..59e8010944e7 100644 --- a/.github/workflows/testBuild.yml +++ b/.github/workflows/testBuild.yml @@ -34,6 +34,10 @@ on: description: Force a full native build, bypassing Rock remote cache type: boolean default: false + MOBILE_EXPENSIFY_MAIN: + description: Build against the tip of Mobile-Expensify main instead of the commit pinned by the submodule. Misses the Rock remote cache. + type: boolean + default: false jobs: # Validates the actor and that they reviewed the PR before any token is used or untrusted code is checked out. @@ -62,8 +66,10 @@ jobs: # Fall back to App main when nothing is resolved (test builds default to the latest NewDot). APP_REF: ${{ needs.resolveRefs.outputs.APP_REF || 'main' }} APP_PR_NUMBER: ${{ needs.resolveRefs.outputs.APP_PR_NUMBER }} - # Fall back to Mobile-Expensify main when nothing is resolved (test builds track the latest OldDot). - MOBILE_EXPENSIFY_REF: ${{ needs.resolveRefs.outputs.MOBILE_EXPENSIFY_REF || 'main' }} + # Empty means the pinned submodule commit, which keeps the Rock fingerprint a function of the App commit + # alone and therefore shared with the builds `testBuildOnPush.yml` seeds the cache from. MOBILE_EXPENSIFY_MAIN + # opts out for the rare case that needs unpinned OldDot. + MOBILE_EXPENSIFY_REF: ${{ needs.resolveRefs.outputs.MOBILE_EXPENSIFY_REF || (inputs.MOBILE_EXPENSIFY_MAIN && 'main' || '') }} MOBILE_EXPENSIFY_PR: ${{ needs.resolveRefs.outputs.MOBILE_EXPENSIFY_PR }} BUILD_WEB: ${{ inputs.WEB && 'true' || 'false' }} BUILD_IOS: ${{ inputs.IOS && 'true' || 'false' }} From 19f40bd2a111e7726d2860e98d575da7cad00bf4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Miko=C5=82aj=20Poche=C4=87?= Date: Fri, 4 Sep 2026 20:51:35 +0200 Subject: [PATCH 2/2] Cut down AI comment --- .github/workflows/testBuild.yml | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/.github/workflows/testBuild.yml b/.github/workflows/testBuild.yml index 59e8010944e7..68b504627345 100644 --- a/.github/workflows/testBuild.yml +++ b/.github/workflows/testBuild.yml @@ -66,9 +66,7 @@ jobs: # Fall back to App main when nothing is resolved (test builds default to the latest NewDot). APP_REF: ${{ needs.resolveRefs.outputs.APP_REF || 'main' }} APP_PR_NUMBER: ${{ needs.resolveRefs.outputs.APP_PR_NUMBER }} - # Empty means the pinned submodule commit, which keeps the Rock fingerprint a function of the App commit - # alone and therefore shared with the builds `testBuildOnPush.yml` seeds the cache from. MOBILE_EXPENSIFY_MAIN - # opts out for the rare case that needs unpinned OldDot. + # Empty means the pinned submodule commit, so the fingerprint matches the builds `testBuildOnPush.yml` seeds the cache from. MOBILE_EXPENSIFY_REF: ${{ needs.resolveRefs.outputs.MOBILE_EXPENSIFY_REF || (inputs.MOBILE_EXPENSIFY_MAIN && 'main' || '') }} MOBILE_EXPENSIFY_PR: ${{ needs.resolveRefs.outputs.MOBILE_EXPENSIFY_PR }} BUILD_WEB: ${{ inputs.WEB && 'true' || 'false' }}