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

simple plan impl #69

Open
wants to merge 1 commit into
base: master
from
Open

simple plan impl #69

wants to merge 1 commit into from

Conversation

@Mikulas
Copy link
Contributor

Mikulas commented Apr 22, 2017

Closes #61

Intentionally very simple implementation. More complex would require many changes to the core, such as splitting OrderResolver. This was started at https://github.com/Mikulas/migrations/tree/pr/exec-plan.

This proposed solution is almost backwards compatible, with single exception of extending existing IConfiguration with getPlanFile method. However, the method is allowed to return NULL, in which case migrations completely skip all Plan code.

Plan entries are created only with migrations:create symfony command. The Plan is checked for validity whenever Engine\Runner::run is invoked.

IMO this solves the merge/rebase problem this was initially intended for, with minimal changes to the codebase.

screenshot 2017-04-22 14 10 29

@hrach hrach requested a review from JanTvrdik Apr 22, 2017
@Mikulas
Copy link
Contributor Author

Mikulas commented Jul 27, 2017

Ok so this above is my proposition. Any better implementation would require a pretty much complete rewrite of the core.

@JanTvrdik JanTvrdik force-pushed the nextras:master branch 3 times, most recently from d0ab005 to 36fd4b7 Oct 2, 2017
@JanTvrdik JanTvrdik force-pushed the nextras:master branch from 5d167b8 to 5970eda Sep 25, 2018
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

1 participant
You can’t perform that action at this time.