Skip to content

chore: codeowners update for token details#25440

Merged
bergarces merged 1 commit intomainfrom
token-details-codeowners-update
Jan 30, 2026
Merged

chore: codeowners update for token details#25440
bergarces merged 1 commit intomainfrom
token-details-codeowners-update

Conversation

@bergarces
Copy link
Copy Markdown
Contributor

@bergarces bergarces commented Jan 30, 2026

Description

Code owners update for new token details page.

Changelog

CHANGELOG entry: null

Related issues

Fixes:

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

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: only updates CODEOWNERS review ownership with no runtime code changes; impact is limited to PR review routing/enforcement for TokenDetails.

Overview
Updates .github/CODEOWNERS to add app/components/UI/TokenDetails under Assets Team (@MetaMask/metamask-assets), ensuring code owner review is required for changes in that UI area.

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

@bergarces bergarces requested a review from a team as a code owner January 30, 2026 13:59
@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.

@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: 98%
click to see 🤖 AI reasoning details

E2E Test Selection:
Only .github/CODEOWNERS was modified to add an ownership entry for app/components/UI/TokenDetails. This has no effect on application behavior, UI, controllers, Engine, test infrastructure, or CI workflows. Safe to skip all E2E tests for this PR.

Performance Test Selection:
No runtime or UI changes; performance is unaffected. No performance tests needed.

View GitHub Actions results

@sonarqubecloud
Copy link
Copy Markdown

@bergarces bergarces changed the title codeowners update chore: codeowners update for token details Jan 30, 2026
@bergarces bergarces enabled auto-merge January 30, 2026 15:09
@bergarces bergarces added this pull request to the merge queue Jan 30, 2026
Merged via the queue into main with commit 3b5ab86 Jan 30, 2026
78 of 86 checks passed
@bergarces bergarces deleted the token-details-codeowners-update branch January 30, 2026 15:44
@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 2026
@metamaskbot metamaskbot added the release-7.65.0 Issue or pull request that will be included in release 7.65.0 label Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.65.0 Issue or pull request that will be included in release 7.65.0 size-XS team-assets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants