-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Rename migration.enabled to migration.6_to_7.enabled #11286
Conversation
This will need a rebase and update as soon as #11284 is merged. Changelog entry should be added. |
This is to make it more clear this is for the migration from 6.7 to 7.0.
943788d
to
b87781a
Compare
I'd keep This option could be called |
I agree with @jsoriano. If his proposal is too verbose, perhaps |
No strong opinions on naming. I'm going with |
Failing test seems related to migration. |
Yes, already fixed, missed one renaming. |
@dedemorton FYI, the config name changed. |
This is to make it more clear this is for the migration from 6.7 to 7.0. (cherry picked from commit 38c81eb)
This is to make it more clear this is for the migration from 6.7 to 7.0.
This is to make it more clear this is for the migration from 6.7 to 7.0.