Skip to content

ci: fix setup-e2e-env tag in mobile workflows#26759

Merged
jvbriones merged 1 commit into
mainfrom
jvbriones/mobile-ci-build-failures-59bd
Mar 2, 2026
Merged

ci: fix setup-e2e-env tag in mobile workflows#26759
jvbriones merged 1 commit into
mainfrom
jvbriones/mobile-ci-build-failures-59bd

Conversation

@jvbriones

@jvbriones jvbriones commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Fixes mobile CI build failures caused by invalid action tag references in workflow files.

Several workflows were using MetaMask/github-tools/.github/actions/setup-e2e-env@v1.7, but v1.7 does not exist in MetaMask/github-tools (valid refs include v1 and v1.7.0).
This PR updates those references to @v1 to restore build job setup resolution.

Changelog

CHANGELOG entry: null

Related issues

Fixes: N/A

Manual testing steps

Feature: CI workflow setup action reference

  Scenario: Build jobs resolve setup-e2e-env action
    Given workflow files reference setup-e2e-env@v1
    When CI runs Build Android E2E APKs and Build iOS E2E Apps jobs
    Then job setup resolves the action successfully
    And CI no longer fails with "unable to find version v1.7"

@cursoragent @jvbriones
ci: restore setup-e2e-env major tag reference
f9bc875 
Co-authored-by: jvbriones <jvbriones@users.noreply.github.com>
@cursor

cursor Bot commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@metamaskbot metamaskbot added the team-qa QA team label Mar 2, 2026
@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: SmokeAccounts
  • Selected Performance tags: None (no tests recommended)
  • Risk Level: low
  • AI Confidence: 85%
click to see 🤖 AI reasoning details

E2E Test Selection:
All 5 changed files are GitHub workflow files with identical changes: updating the version reference for the external MetaMask/github-tools/.github/actions/setup-e2e-env action from @v1.7 to @v1. This is a CI infrastructure change that affects the E2E test environment setup across Android E2E builds, iOS E2E builds, the main build workflow, iOS Flask smoke tests, and the main E2E workflow runner.

Since this is purely a CI infrastructure change with no modifications to application code, test logic, or functional behavior, the risk to application functionality is low. However, because this change affects the E2E test infrastructure itself, running a minimal smoke test (SmokeAccounts) will help validate that the CI pipeline and E2E environment setup still works correctly with the updated action version.

SmokeAccounts was selected as a minimal validation test because:

  1. It covers core wallet functionality (account management)
  2. It's a relatively quick test suite
  3. It will validate that the E2E environment setup works correctly

No performance tests are needed as this change doesn't affect any application code or UI rendering.

Performance Test Selection:
No performance tests needed. This PR only changes CI workflow files to update an external GitHub Action version reference (v1.7 → v1). No application code, UI components, or performance-critical paths are modified.

View GitHub Actions results

@jvbriones jvbriones marked this pull request as ready for review March 2, 2026 10:12
@jvbriones jvbriones requested a review from a team as a code owner March 2, 2026 10:12
@christopherferreira9 christopherferreira9 changed the title Mobile CI build failures ci: Mobile CI build failures Mar 2, 2026
@christopherferreira9 christopherferreira9 added the no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Mar 2, 2026
@metamaskbot metamaskbot added the INVALID-PR-TEMPLATE PR's body doesn't match template label Mar 2, 2026
@jvbriones jvbriones changed the title ci: Mobile CI build failures ci: fix setup-e2e-env tag in mobile workflows Mar 2, 2026
@jvbriones jvbriones enabled auto-merge March 2, 2026 10:23
@jvbriones jvbriones added this pull request to the merge queue Mar 2, 2026
Merged via the queue into main with commit bee9243 Mar 2, 2026
121 of 128 checks passed
@jvbriones jvbriones deleted the jvbriones/mobile-ci-build-failures-59bd branch March 2, 2026 11:14
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 2, 2026
@metamaskbot metamaskbot added the release-7.69.0 Issue or pull request that will be included in release 7.69.0 label Mar 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@tommasini tommasini tommasini approved these changes

@christopherferreira9 christopherferreira9 christopherferreira9 approved these changes

Assignees

No one assigned

Labels

INVALID-PR-TEMPLATE PR's body doesn't match template no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-7.69.0 Issue or pull request that will be included in release 7.69.0 size-XS team-qa QA team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jvbriones @tommasini @christopherferreira9 @metamaskbot @cursoragent