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)
Wed, May 15, 5:27 AM
Unknown Object (File)
Mon, Apr 22, 7:24 PM
Unknown Object (File)
Mon, Apr 22, 7:24 PM
Unknown Object (File)
Mon, Apr 22, 7:24 PM
Unknown Object (File)
Mon, Apr 22, 7:21 PM
Unknown Object (File)
Apr 7 2024, 9:16 PM
Unknown Object (File)
Mar 20 2024, 1:27 PM
Unknown Object (File)
Mar 2 2024, 7:59 PM

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.