Skip to content

chore: Remove all bitrise artifacts#30204

Merged
Cal-L merged 5 commits into
mainfrom
chore/remove-bitrise
May 14, 2026
Merged

chore: Remove all bitrise artifacts#30204
Cal-L merged 5 commits into
mainfrom
chore/remove-bitrise

Conversation

@Cal-L
Copy link
Copy Markdown
Contributor

@Cal-L Cal-L commented May 14, 2026

Description

This PR completely removes Bitrise from the app

Changelog

CHANGELOG entry:

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/MCWP-260

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

After

Pre-merge author checklist

Performance checks (if applicable)

  • I've tested on Android
    • Ideally on a mid-range device; emulator is acceptable
  • I've tested with a power user scenario
    • Use these power-user SRPs to import wallets with many accounts and tokens
  • I've instrumented key operations with Sentry traces for production performance metrics

For performance guidelines and tooling, see the Performance Guide.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Medium Risk
Medium risk because it removes Bitrise-triggered workflows/scripts and related build metadata handling, which could disrupt CI/build/release pipelines if any consumers still depend on them.

Overview
Removes Bitrise-specific CI/E2E tooling and cleanup of remaining references, shifting fully to GitHub Actions-based build workflows.

This deletes the Bitrise PR label/E2E gate scripts under .github/scripts/bitrise/, drops the associated workflow YAMLs (previously .disabled), and prunes Bitrise-only dependencies from .github/scripts/package.json/yarn.lock. It also removes Bitrise config hooks from .detoxrc.js, Gradle (Bitrise NDK path helper), version-bump workflow (update-latest-build-version.yml no longer stages bitrise.yml), and updates docs/comments/config filters/CODEOWNERS to stop treating bitrise.yml as a critical/owned/CI file.

Reviewed by Cursor Bugbot for commit 4cc0ebf. Bugbot is set up for automated code reviews on this repo. Configure here.

@Cal-L Cal-L requested review from a team as code owners May 14, 2026 19:18
@github-actions
Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbotv2 metamaskbotv2 Bot added the team-mobile-platform Mobile Platform team label May 14, 2026
@Cal-L Cal-L added No QA Needed Apply this label when your PR does not need any QA effort. needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed no changelog required No changelog entry is required for this change and removed team-mobile-platform Mobile Platform team labels May 14, 2026
@github-project-automation github-project-automation Bot moved this to Needs dev review in PR review queue May 14, 2026
@Cal-L Cal-L added the team-mobile-platform Mobile Platform team label May 14, 2026
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit db73661. Configure here.

Comment thread .github/scripts/package.json
tommasini
tommasini previously approved these changes May 14, 2026
weitingsun
weitingsun previously approved these changes May 14, 2026
cortisiko
cortisiko previously approved these changes May 14, 2026
@Cal-L
Copy link
Copy Markdown
Contributor Author

Cal-L commented May 14, 2026

E2E test failure not related to changes

@Cal-L
Copy link
Copy Markdown
Contributor Author

Cal-L commented May 14, 2026

Running E2E performance tests - https://github.com/MetaMask/metamask-mobile/actions/runs/25884408647

@Cal-L Cal-L dismissed stale reviews from cortisiko and weitingsun via 4cc0ebf May 14, 2026 21:17
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

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

E2E Test Selection:
This PR is a Bitrise CI/CD migration cleanup. All changes are infrastructure-only with no functional app code modifications:

  1. bitrise.yml deleted (3,608 lines) — Removes the entire Bitrise CI configuration file. This was the old CI system being decommissioned. No app code affected.

  2. .detoxrc.js — Removes the android.bitrise.emulator device configuration. This device config was only used on Bitrise runners (which are being removed). The GHA-based E2E pipeline uses different device configs that remain intact.

  3. GitHub workflow files — All changes are comment/label text updates only (e.g., "Bitrise" → "BrowserStack", removing "on Bitrise" from job names). No functional workflow logic changed.

  4. App source files (ramps-service-init.ts, rewards-api-url.ts, getSdkEnvironment.ts, mapBaanxApiUrl.ts) — All comment-only changes updating JSDoc comments from "Bitrise / .js.env / E2E" to "legacy .js.env / E2E". Zero functional code changes.

  5. tests/page-objects/Settings/NetworksView.ts — Single comment change: "bitrise" → "CI". No test logic changed.

  6. package.json — Removes build:announce script (Bitrise notification script) and removes axios and simple-git dependencies from .github/scripts/package.json. These are CI tooling dependencies, not app dependencies.

  7. Test scripts — Cleanup of Bitrise references in shell scripts.

  8. fast-analyzer.ts — Removes bitrise/ from exclude patterns (since the directory no longer exists).

None of these changes affect any user-facing functionality, test logic, app behavior, navigation, controllers, or E2E test infrastructure that would require running E2E tests. The .detoxrc.js change only removes a device config that was never used in the current GHA pipeline. No performance-sensitive code was modified.

Performance Test Selection:
No performance-sensitive code was changed. All modifications are CI/CD infrastructure cleanup (removing Bitrise configuration), comment-only changes in app source files, and script cleanup. No UI components, data loading, state management, or critical user flows were modified.

View GitHub Actions results

@sonarqubecloud
Copy link
Copy Markdown

@Cal-L Cal-L merged commit 3a5db6d into main May 14, 2026
61 of 62 checks passed
@github-project-automation github-project-automation Bot moved this from Needs dev review to Merged, Closed or Archived in PR review queue May 14, 2026
@Cal-L Cal-L deleted the chore/remove-bitrise branch May 14, 2026 21:47
@github-actions github-actions Bot removed the needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) label May 14, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators May 14, 2026
@metamaskbotv2 metamaskbotv2 Bot added the release-7.79.0 Issue or pull request that will be included in release 7.79.0 label May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no changelog required No changelog entry is required for this change No QA Needed Apply this label when your PR does not need any QA effort. no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-7.79.0 Issue or pull request that will be included in release 7.79.0 size-XL team-mobile-platform Mobile Platform team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants