-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: split migrations into multiple files #7873
Conversation
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. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #7873 +/- ##
=======================================
Coverage 36.60% 36.60%
=======================================
Files 1092 1120 +28
Lines 29178 29178
Branches 2677 2678 +1
=======================================
Hits 10681 10681
Misses 17885 17885
Partials 612 612 ☔ View full report in Codecov by Sentry. |
57bd2ed
to
2868d1b
Compare
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/07d34fed-5d1b-44ed-b8e4-a3267d112211 |
61fc24d
to
030c846
Compare
001dae0
to
39d2f6d
Compare
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/8bde408d-69af-453e-8898-2d9e03dd3cdf |
Kudos, SonarCloud Quality Gate passed! |
Description
Split each migration function and the associated unit tests into separate files.
See the extension approach which also exports the migration version from each individual migration file (example)
Related issues
Fixes: #7845
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist