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
[Spring cleaning] Refactor migrations #1737
base: 1.x
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thanks @adevade @glennjacobs This looks like something we should get in (if wanted) before too much else gets merged to give us a good baseline. |
Maybe run this on an existing site to ensure nothing weird happens before merging? |
@glennjacobs Yeah, I have not tested for that. I don't believe there should be any problems though. |
Just ran this on the starter kit and didn't come against any issues so if everyone is happy @glennjacobs I guess it can go in. |
Note: This PR is probably easiest reviewed commit by commit instead of all changes at once.
Just some spring cleaning and standardization of the migration classes. I converted all to anonymous classes, like Laravel did during the 10.x releases.
Removed PHPDoc blocks, since they were not always there and less clutter.
All tests still passing locally.
Feel free to request changes or revert single commits if you don't like some of the changes.