Skip to content

Add more documentation about DIC core concepts #2014

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

Closed
fabpot opened this issue Dec 11, 2012 · 5 comments
Closed

Add more documentation about DIC core concepts #2014

fabpot opened this issue Dec 11, 2012 · 5 comments

Comments

@fabpot
Copy link
Member

fabpot commented Dec 11, 2012

see symfony/symfony#6218

@richardmiller-zz
Copy link
Contributor

@sun So from symfony/symfony#6218 it looks like scope and synthetic services are the main things missing. With scope it is how to actual implement different scopes as opposed to just the end user info on the different scopes of the component and framework.

Is there anything else missing from the component documentation that would be good to see from an implementers point of view?

@weaverryan
Copy link
Member

Related: Fabien has updated scope and covered synchronized services in #2343

@richardmiller-zz
Copy link
Contributor

I have started something on scope for the DI component section and managed to make some progress with it last week but it still needs some gaps filling in

@wouterj
Copy link
Member

wouterj commented Jun 5, 2013

@richardmiller do you have any update about this one?

@weaverryan
Copy link
Member

I think we can probably close this now. The 2 big items - scopes and synthetic services - are now definitely covered in the documentation. In fact, thanks to a lot of work done by Richard, I'd be surprised if we were really missing any of the core concepts in the DI component docs. I'm sure there's room for improvement to make the whole component easier to understand, but I think we have good coverage.

If anyone can think of some specific items that are still missing, let's open up a new issue for these.

Thanks everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants