Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an 'ignorePendingMigrations' configuration option #2129

Merged
merged 2 commits into from Sep 19, 2018

Conversation

@yjo
Copy link
Contributor

@yjo yjo commented Aug 31, 2018

This adds an 'ignorePendingMigrations' option to suppress validation errors caused by the availability of not-yet-applied migrations (feature request #2128).

My use case is for a CI/CD pipeline - while developers are working on a feature branch they should be able to create and modify migrations for that feature. However, we should be able to detect and break the build when changes are made to versioned migrations that have already been applied to (e.g.) our production database. Running flyway validate with ignorePendingMigrations=true supports this scenario and is consistent with the existing ignore*Migrations options.

@CLAassistant
Copy link

@CLAassistant CLAassistant commented Aug 31, 2018

CLA assistant check
All committers have signed the CLA.

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Aug 31, 2018

Not necessary. See #2128

@axelfontaine axelfontaine merged commit 498fba7 into flyway:master Sep 19, 2018
2 checks passed
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Sep 19, 2018

Merged. I had to make some minor adjustments as you had missed a few places where this property had to be carried over (it's not easy to catch them all). Thanks for the PR and thanks for detailing your use case in #2128 .

dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
Add an 'ignorePendingMigrations' configuration option
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants