-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Create schemas and permissions in more situations #2838
Conversation
On trusty, this is needed for all mysql/mariadb builds. Fixes doctrine#2837
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, we don't really want non-determinism on CI.
Good enough until we migrate DBAL to Build Stages.
Restarting the build because I've had a new issue locally even with that change |
Ok still green :) |
I know, but your PR is big and still red, so not mergeable right now. I just noticed that Travis is not required for the build to pass, so this is not as critical as I think. However this situation is not great, because people might assume they didn't break the build when they actually did. Right now they can't know for sure unless they check the output of the logs on each commit. |
Handled in #2849, sorry for extra work. |
Not a big deal :) |
On trusty, this is needed for all mysql/mariadb builds.
Fixes #2837