ci: simplify migration test matrix - #1347
piotr-iohk wants to merge 2 commits into
Conversation
|
Fresh E2E migration validation: https://github.com/synonymdev/bitkit-android/actions/runs/36161732548 |
Regtest APKDownload bitkit-dev-debug universal APK (expires in 30 days). |
|
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
|
Clarification on the migration run above: the only failing case, This branch does not contain the fix merged in #1343. The other three migration cases passed. Once this branch includes #1343 (for example by updating it from current |
pwltr
left a comment
There was a problem hiding this comment.
Reviewed 1b58ce70f0e2caa1498b59b39359ee88dec651b5 with the companion migration changes.
No actionable code findings. The earlier shell-quoting concern is addressed by passing MIGRATION_SOURCE and MIGRATION_VERSION through environment variables.
Build, lint and detekt pass; migration jobs were skipped on this head. The earlier manual run passed three cases and failed rn_restore. The author attributes that failure to #1342 and notes that this branch lacks the fix merged in #1343. This is an existing product regression, not a new finding in this workflow change.
Include that fix and rerun migration coverage before considering verification complete. Merge E2E #258 first as documented in the coordinated rollout.
Description
Design
N/A — no UI changes.
Preview
N/A
QA Notes
Journeys
N/A — no user-visible behaviour change.
Manual Tests
N/A
Automated Checks
e2e_migration.yml— runs four routine migration paths with independently resolved source artifacts and retries.Coordinated rollout
This PR is part of one migration-test rollout and should be reviewed with:
Merge the E2E PR first. Before merging the iOS PR, change its reusable wallet-setup workflow reference from the temporary companion branch to
@main. Then merge the Android and iOS workflow PRs.