Page MenuHomePhabricator

[native] Add support for async migrations
ClosedPublic

Authored by michal on Jun 28 2023, 4:11 AM.
Tags
None
Referenced Files
F2110581: D8353.id28207.diff
Tue, Jun 25, 8:20 PM
F2109888: D8353.id28572.diff
Tue, Jun 25, 5:56 PM
Unknown Object (File)
Mon, Jun 24, 4:04 PM
Unknown Object (File)
Mon, Jun 24, 4:04 PM
Unknown Object (File)
Mon, Jun 24, 4:03 PM
Unknown Object (File)
Mon, Jun 24, 3:54 PM
Unknown Object (File)
Wed, Jun 19, 6:22 PM
Unknown Object (File)
Wed, Jun 12, 8:23 PM
Subscribers

Details

Summary
Test Plan

Add a few interspersed async and sync dummy migration and check if they worked.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

It will make sense to add a test case when you check older, already-written migrations with createAsyncMigrate. It doesn't make a huge difference but it's a fragile part so additional testing could profit

This revision is now accepted and ready to land.Jun 29 2023, 3:59 AM