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

SimpleContainer property injection can be recursive #468

Closed
Stannieman opened this issue Aug 14, 2017 · 1 comment
Closed

SimpleContainer property injection can be recursive #468

Stannieman opened this issue Aug 14, 2017 · 1 comment
Milestone

Comments

@Stannieman
Copy link

Stannieman commented Aug 14, 2017

BuildUp can call BuildUp again for every injected property to build up the whole graph instead of only the instance passed into it.

EDIT: PR created.

@nigel-sampson
Copy link
Contributor

I've addressed this in #471 in order to keep discussion in one place.

nigel-sampson pushed a commit that referenced this issue Jul 15, 2019
@nigel-sampson nigel-sampson added this to the v4.0.0 milestone Jul 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants