Skip to content

[Doctrine] Update doctrine.rst #14572

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 20, 2020
Merged

[Doctrine] Update doctrine.rst #14572

merged 1 commit into from
Nov 20, 2020

Conversation

abdounikarim
Copy link
Contributor

Hello,

I just add env variable for mariadb.

@abdounikarim
Update doctrine.rst
c639ea4 
Hello,

I just add env variable for mariadb.
@carsonbot carsonbot added this to the 4.4 milestone Nov 19, 2020
@carsonbot carsonbot changed the title Update doctrine.rst [Doctrine] Update doctrine.rst Nov 20, 2020
@javiereguiluz
Copy link
Member

I was divided about this: we prefer to only show what the Symfony Recipe creates ... but on the other hand, MariaDB is somewhat popular and its configuration is tricky because you need to use mysql as "protocol" and then add mariadb prefix in the version "number". So, let's merge it. Thanks!

@javiereguiluz javiereguiluz merged commit 901b623 into symfony:4.4 Nov 20, 2020
@abdounikarim abdounikarim deleted the patch-22 branch November 20, 2020 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Doctrine Status: Reviewed
Projects
None yet
Milestone
4.4
Development

Successfully merging this pull request may close these issues.
3 participants
@abdounikarim @javiereguiluz @carsonbot