Skip to content

Refactor pd migrate code to abstract away boilerplate #4294

Open
@aubrika

Description

@aubrika

Currently the migration code features a lot of copy-pasted functions which are common to every migration, and for the purposes of ergonomics, readability and bug surface this is not ideal. This should be kept as fairly straightforward or simple refactor with a mind to facilitating future automated testing of migrations and enhancing testability.

Suggestions from backlog grooming:

  • migrations should use the APP_VERSION, which would give us idempotency for free
  • migrations should be refactored to take a StateWrite

Metadata

Metadata

Assignees

Labels

A-upgradesArea: Relates to chain upgrades_P-lowPriority: low

Type

No type

Projects

Status

Backlog

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions