Add new subcommand to validate migration files #856
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a new subcommand to validate migration files called
validate
.It is first step towards providing a way to validate migration files before running them. At the moment it can validate a single migration file against the latest schema in the database. This can be added to your local workflow, or a linter to make sure that the newly added migration file is correct.
Future improvements can be
up
migrations on a subset of the table Allow for migration dry-runs to verifyup
SQL #169jsonschema.Validate
during validate Use JSON schema validation duringValidate
phase #673The PR contains some minor changes to the migration phase
Start
. Now migration validation is decoupled from it. The previous implementation ofStart
started a new migration before validating the migration it got as a parameter. So this change does not interfere with the UX of pgroll.Closes #370