Page MenuHomePhabricator

[keyserver] Extract migration config to separate file
ClosedPublic

Authored by ashoat on May 30 2022, 1:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 7:55 AM
Unknown Object (File)
Tue, Dec 10, 12:42 PM
Unknown Object (File)
Fri, Nov 22, 12:53 PM
Unknown Object (File)
Fri, Nov 22, 12:53 PM
Unknown Object (File)
Fri, Nov 22, 12:53 PM
Unknown Object (File)
Oct 28 2024, 2:49 AM
Unknown Object (File)
Oct 25 2024, 12:10 AM
Unknown Object (File)
Oct 25 2024, 12:09 AM

Details

Summary

In the following diff, I'm going to check the latest migration index from setup-db.js so we can start with a default migration version.

Without this diff, that change would result in a require cycle.

Depends on D4151

Test Plan

This is a simple refactor. I tested it in combination with stack (see test plan in D4151), and also used Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.May 31 2022, 5:13 AM

Makes sense, moving the "contents" of the migrations to a separate file helps reduce clutter in migrations.js.

At some point it'll probably make sense to have a migrations directory instead of having them all in migration-config.js, but it's probably fine for now.