-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[Updated PR#1476] Add explanation for database migration resets #1941
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
Conversation
Co-Authored-By: theminer3746 <[email protected]>
I think both |
Sure, I'm just trying to update/resolve PRs that doesn't have a breaking change, after that we could focus on actual bugs. Thanks! |
Yep, I saw. Good job @divine |
I have added some tests on demo application here. @divine @Smolevich
in order to avoid other issues to be opened. |
@Smolevich @Giacomo92 @rennokki what you think guys? |
i think that it's ok |
[Updated PR#1476] Add explanation for database migration resets
This PR was recreated from #1476 with updated branch or resolved conflicting files.
--
Add Database Testing to index.
use DatabaseMigrations; instead of use RefreshDatabase; for laravel 5.6 compatibility issue (#1475)
Co-Authored-By: theminer3746 [email protected]