Skip to content
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

Update reverse_engineering.rst #19817

Open
wants to merge 2 commits into
base: 7.1
Choose a base branch
from
Open

Conversation

siburuxue
Copy link

use the doctrine-helper instead of php bin/console doctrine:mapping:import

document: https://github.com/siburuxue/doctrine-helper

use the doctrine-helper instead of php bin/console doctrine:mapping:import 

document: https://github.com/siburuxue/doctrine-helper
@carsonbot carsonbot added this to the 7.1 milestone Apr 22, 2024

.. _`Symfony Maker Bundle`: https://symfony.com/bundles/SymfonyMakerBundle/current/index.html
Copy link
Contributor

Choose a reason for hiding this comment

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

why not keep both, and just in your repo project's readme add such documentation ?
i do not think this is relevant to document much here as the tool is not maintained by sf core ppls, thus maintaining a synchro in code+doc can be very hard here

Copy link
Author

@siburuxue siburuxue Apr 24, 2024

Choose a reason for hiding this comment

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

why not keep both, and just in your repo project's readme add such documentation ? i do not think this is relevant to document much here as the tool is not maintained by sf core ppls, thus maintaining a synchro in code+doc can be very hard here

image

Can this be changed?

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.

None yet

3 participants