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 form_dependencies.rst: fix DI #7009

Closed
wants to merge 1 commit into from
Closed

Update form_dependencies.rst: fix DI #7009

wants to merge 1 commit into from

Conversation

ReDnAxE
Copy link
Contributor

@ReDnAxE ReDnAxE commented Sep 26, 2016

Fix service definitions for __construct(EntityManager $em) argument

Fix service definitions for `__construct(EntityManager $em)` argument
@xabbuh
Copy link
Member

xabbuh commented Sep 27, 2016

👍

Status: Reviewed

@xabbuh
Copy link
Member

xabbuh commented Sep 27, 2016

Good catch, thanks @ReDnAxE.

xabbuh added a commit that referenced this pull request Sep 27, 2016
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #7009).

Discussion
----------

Update form_dependencies.rst: fix DI

Fix service definitions for `__construct(EntityManager $em)` argument

Commits
-------

7dbcb7c Update form_dependencies.rst: fix DI
@xabbuh xabbuh closed this Sep 27, 2016
@ReDnAxE
Copy link
Contributor Author

ReDnAxE commented Sep 27, 2016

@xabbuh sorry, but why this PR has been closed without being merged ?

@HeahDude
Copy link
Contributor

@ReDnAxE This PR says "closed" because it was merged in 2.7 the oldest maintained branch needing that fix but was targeting master.

It will be merged in newer branches from there.

@ReDnAxE
Copy link
Contributor Author

ReDnAxE commented Sep 27, 2016

Ok thank you @HeahDude for the explanation, thank's @xabbuh ^^

@xabbuh
Copy link
Member

xabbuh commented Sep 27, 2016

@ReDnAxE Oh sorry, I didn't want to shock you. But @HeahDude explained our process very well. :) We always merge PRs into the lowest affected branch that is still supported and then frequently merge things up from there to master. So your changes will also be available soon on all other branches. :)

@ReDnAxE
Copy link
Contributor Author

ReDnAxE commented Sep 27, 2016

@xabbuh my fault ;) indeed I knew it !

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

4 participants