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
Fix installer when database not exists #6739
base: master
Are you sure you want to change the base?
Conversation
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.
Maybe you can add an extra build without creating the MySQL or PG database first to see if it fix the problem
app/config/parameters.clean
Outdated
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.
What is that file?
1b33a47
to
a848281
Compare
All is green but the CI is broken 😉 |
@@ -151,3 +151,65 @@ jobs: | |||
|
|||
- name: "Run PHPUnit" | |||
run: "php bin/simple-phpunit -v" | |||
|
|||
phpunit_no_database: |
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.
I'm not a big fan of running these new tests on all PR as it unnecessarily increases load on the CI. @nicosomb could we disable them by default (or drop the commit before merging)?
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.
Maybe we can just keep one? For MySQL or PG
1d2dc8a
to
a2a3bb3
Compare
a2a3bb3
to
e185e03
Compare
@j0k3r can you help me about the new build to create please? |
@nicosomb the build seems ok now, but tests are failing |
I'd prefer have a regex to check the whole exception message. If someone wants to have a look on it.
My PR is a pointer to the bug, we can improve it.
I want to kill an other bug quickly #6659