Skip to content

Doctrine reverse engineering updates for 4.0 #9491

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

Merged
merged 2 commits into from
Apr 18, 2018
Merged

Doctrine reverse engineering updates for 4.0 #9491

merged 2 commits into from
Apr 18, 2018

Conversation

weaverryan
Copy link
Member

@weaverryan weaverryan commented Mar 23, 2018

This is waiting on https://github.com/doctrine/DoctrineBundle#791 to be in a tagged release.

But, otherwise, I just tried the entire process with that patch. Works great.

@weaverryan weaverryan added the Waiting Code Merge Docs for features pending to be merged label Mar 23, 2018
@weaverryan weaverryan changed the title [WCM] Doctrine reverse engineering updates for 4.0 Doctrine reverse engineering updates for 4.0 Apr 18, 2018
@weaverryan weaverryan removed the Waiting Code Merge Docs for features pending to be merged label Apr 18, 2018
@weaverryan weaverryan merged commit 61bdd26 into symfony:4.0 Apr 18, 2018
weaverryan added a commit that referenced this pull request Apr 18, 2018
This PR was squashed before being merged into the 4.0 branch (closes #9491).

Discussion
----------

Doctrine reverse engineering updates for 4.0

This is waiting on https://github.com/doctrine/DoctrineBundle#791 to be in a tagged release.

But, otherwise, I just tried the entire process with that patch. Works great.

Commits
-------

61bdd26 Updating arg: we need to pass the full namespace (with double slashes)
e8eb547 updating the doctrine reverse engineering for 4.0
@weaverryan weaverryan deleted the doctrine-reverse-engineer branch April 18, 2018 17:50
@xabbuh xabbuh added this to the 4.0 milestone Apr 19, 2018
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.

3 participants