Skip to content

[Doctrine] update config reference doc with PHP examples #19596

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

vinceAmstoutz
Copy link
Contributor

Complete the Doctrine Configuration Reference (DoctrineBundle) documentation by adding the equivalent PHP configurations wich are missing.

@carsonbot carsonbot added this to the 6.4 milestone Feb 24, 2024
@vinceAmstoutz
Copy link
Contributor Author

@javiereguiluz As we discussed on Slack, I created this PR to complete the doc

@vinceAmstoutz vinceAmstoutz force-pushed the improve_doctrine_config_reference_by_adding_php_synthax branch from cc0d29e to 1da385a Compare June 12, 2025 07:56
Copy link
Member

@alexandre-daubois alexandre-daubois left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice addition, always in favor of pushing PHP config forward! Could we merge this one @symfony/docs if it's fine for you? 🙂

@vinceAmstoutz vinceAmstoutz changed the title Updates Doctrine configuration reference documentation with PHP configuration examples [Doctrine] update config reference doc with PHP configuration examples Jun 16, 2025
@vinceAmstoutz vinceAmstoutz changed the title [Doctrine] update config reference doc with PHP configuration examples [Doctrine] update config reference doc with PHP examples Jun 16, 2025
@vinceAmstoutz
Copy link
Contributor Author

@javiereguiluz @OskarStark

@javiereguiluz javiereguiluz force-pushed the improve_doctrine_config_reference_by_adding_php_synthax branch from 8653de4 to 29106ac Compare June 19, 2025 14:48
@javiereguiluz
Copy link
Member

I'm so sorry we forgot about this PR 🙏 Thanks for the ping.

This is now merged. Thanks a lot Vincent!

@javiereguiluz javiereguiluz merged commit 1631885 into symfony:6.4 Jun 19, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants