-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Adding a note describing the configuration of SQLite with doctrine #2196
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
Can you please add the pull request format to your PR message next time? |
dbal: | ||
driver: pdo_sqlite | ||
path: %kernel.root_dir%/sqlite.db | ||
charset: UTF8 |
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.
You should also add a XML and PHP example:
<!-- app/config/config.xml -->
<doctrine:config
driver="pdo_sqlite"
path="%kernel.root_dir%/sqlite.db"
charset="UTF-8"
>
<!-- ... -->
</doctrine:config>
// app/config/config.php
$container->loadFromExtension('doctrine', array(
'dbal' => array(
'driver' => 'pdo_sqlite',
'path' => '%kernel.root_dir%/sqlite.db',
'charset' => 'UTF-8',
),
));
I changed the PR description. I didn't knew about this new format. |
'path' => '%kernel.root_dir%/sqlite.db', | ||
'charset' => 'UTF-8', | ||
), | ||
)); |
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.
You should put the code blocks in a configuration block:
.. configuration-block::
.. code-block:: yml
# ...
.. code-block:: xml
<!-- ... -->
.. code-block:: php
// ...
Adding a note describing the configuration of SQLite with doctrine
… has so many notes and sidebars already
Thanks Sebastian and Wouter! |
As discussed in #1638, the doctrine configuration option for SQLite was added as a note (including the correct configuration for the driver and the encoding setting).