Skip to content

chore: update CODEOWNERS to @MetaMask/money-movement#28131

Merged
amitabh94 merged 2 commits into
mainfrom
chore/codeowners-money-movement
Mar 31, 2026
Merged

chore: update CODEOWNERS to @MetaMask/money-movement#28131
amitabh94 merged 2 commits into
mainfrom
chore/codeowners-money-movement

Conversation

@amitabh94

@amitabh94 amitabh94 commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

Description

The Ramps lane has been renamed to Money Movement on GitHub (@MetaMask/money-movement). This updates .github/CODEOWNERS so review requests and branch-protection ownership resolve to the current org team instead of @MetaMask/ramp.

What changed

  • Section comment renamed to Money Movement Team (formerly Ramps).
  • All @MetaMask/ramp entries replaced with @MetaMask/money-movement (Ramp UI, fiat orders, ramps controller/messengers, selectors, path globs, and Swaps co-owned parseAmount / parseAmount.test.ts).
  • Co-ownership comment updated to reference Money Movement alongside Swaps.

Changelog

CHANGELOG entry: null

Related issues

Fixes:

Manual testing steps

Feature: CODEOWNERS ownership update

  Scenario: No in-app verification required
    Given this pull request only changes .github/CODEOWNERS
    When reviewers validate the team slug and paths
    Then no wallet UI or device manual test is applicable

Screenshots/Recordings

Before

N/A

After

N/A

Pre-merge author checklist

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

Low Risk
Low risk because it only updates .github/CODEOWNERS mappings; the main impact is on review routing/branch protection if any paths or the new team slug are incorrect.

Overview
Updates .github/CODEOWNERS to reflect the GitHub team rename from @MetaMask/ramp to @MetaMask/money-movement for Ramp-related paths (UI, fiat orders, ramps controllers/messengers, selectors, and related glob patterns).

Adds an additional **/money-movement/** ownership rule and updates the Swaps co-owned parseAmount entries to use the new Money Movement team.

Written by Cursor Bugbot for commit 8efd4d2. This will update automatically on new commits. Configure here.

@amitabh94 amitabh94 self-assigned this Mar 31, 2026
@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.

@metamaskbot metamaskbot added the team-money-movement issues related to Money Movement features label Mar 31, 2026
@amitabh94 amitabh94 marked this pull request as ready for review March 31, 2026 00:54
@amitabh94 amitabh94 requested a review from a team as a code owner March 31, 2026 00:54
@github-actions github-actions Bot added the risk-low Low testing needed · Low bug introduction risk label Mar 31, 2026
@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: 99%
click to see 🤖 AI reasoning details

E2E Test Selection:
The only changed file is .github/CODEOWNERS, which is a GitHub repository metadata file that defines code ownership for PR review assignments. The change renames the @MetaMask/ramp team to @MetaMask/money-movement across all relevant path patterns, adds a new **/money-movement/** path entry, and updates comments accordingly. This is a pure organizational/administrative change with zero impact on:

  • Application source code
  • Test infrastructure or test files
  • Build configuration
  • Runtime behavior
  • Any user-facing functionality

No E2E tests are needed since no application code, test framework, or CI pipeline logic was modified. This change only affects who gets auto-assigned as reviewers on future PRs.

Performance Test Selection:
The CODEOWNERS change has no impact on application performance. No code paths, rendering logic, data loading, or state management were modified.

View GitHub Actions results

@sonarqubecloud

Copy link
Copy Markdown

@amitabh94 amitabh94 enabled auto-merge March 31, 2026 01:41
amitabh94 added a commit that referenced this pull request Mar 31, 2026
@amitabh94 amitabh94 added this pull request to the merge queue Mar 31, 2026
Merged via the queue into main with commit b02b848 Mar 31, 2026
67 checks passed
@amitabh94 amitabh94 deleted the chore/codeowners-money-movement branch March 31, 2026 03:55
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 31, 2026
@weitingsun weitingsun added release-7.73.0 Issue or pull request that will be included in release 7.73.0 and removed release-100.10.0 labels Mar 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.73.0 Issue or pull request that will be included in release 7.73.0 risk-low Low testing needed · Low bug introduction risk size-S team-money-movement issues related to Money Movement features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants