MetaMask / MetaMask/metamask-extension

ci - update `identify-codeowners` to use the team topology file (source of truth)

Open
#37,360 2 comments 0 reactions 1 assignee Claimed by @gauthierpetetin View on GitHub
needs-triage team-extension-platform
Dominant language
TypeScript
Stars
13.2k
Forks
5.6k
Avg merge
2d 5h
Merged PRs (30d)
451

Description

### What is this about?

The identify-codeowners logic uses a [hardcoded team list](https://github.com/MetaMask/metamask-extension/blob/main/.github/scripts/identify-codeowners.ts#L27). To avoid maintenance issues, I think we should:

- Remove the hardcoded list and use the new team topology file
and
- The identify-codeowners list also attaches an emoji to each team. Should we add an emoji for each team, in the new team topology file?

### Scenario

_No response_

### Design

_No response_

### Technical Details

_No response_

### Threat Modeling Framework

_No response_

### Acceptance Criteria

_No response_

### Stakeholder review needed before the work gets merged

- [ ] Engineering (needed in most cases)
- [ ] Design
- [ ] Product
- [ ] QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
- [ ] Security
- [ ] Legal
- [ ] Marketing
- [ ] Management (please specify)
- [ ] Other (please specify)

### References

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.