MetaMask / MetaMask/metamask-mobile

Implement a CI action that flags pull requests requiring migrations.

Open
#6,629 1 comment 0 reactions 0 assignees View on GitHub
type-enhancement
Dominant language
TypeScript
Stars
3k
Forks
1.7k
Avg merge
1d 14h
Merged PRs (30d)
669

Description

**Describe the usability problem**

We currently do not have the ability to flag whenever a PR needs migrations for a release. Sometimes this might lead to bugs when upgrading between app versions. As our team expands, it becomes essential to establish a streamlined process for automatically identifying PRs that require migrations, eliminating the need for manual effort.

**Screenshots**
_If applicable, add screenshots or links to help explain the issue_

**Proposed behavior**
There are many solutions to this but I propose we have CI checks for migrations:

### At the feature level:

After a new feature has undergone development, a CI check can be implemented to determine whether a ticket requires migrations.

OR
### At the release level:

Implementing a check during the process of cutting an RC to identify whether PRs require migrations or not. This additional step would help in accurately determining the migration needs associated with each PR.

Contributor guide

Open the contributing guide

Research direction

Compare the proposed feature-level and release-level approaches, then inspect the repository's CI and release workflows to determine where such a check could run. Define how a pull request or release would be identified as requiring migrations and what passing or flagged output should look like; the issue does not name specific files or tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions
Domain
ci-cd, release
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.