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)
Sun, Feb 23, 6:11 AM
Unknown Object (File)
Jan 31 2025, 2:20 PM
Unknown Object (File)
Jan 16 2025, 10:47 AM
Unknown Object (File)
Jan 15 2025, 2:09 PM
Unknown Object (File)
Jan 13 2025, 8:47 AM
Unknown Object (File)
Jan 13 2025, 8:47 AM
Unknown Object (File)
Jan 13 2025, 8:47 AM
Unknown Object (File)
Jan 13 2025, 5:08 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
No Lint Coverage
Unit
No Test Coverage

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.