search

tomphp / container-configurator

Configure your application and the Dependency Injection Container (DIC) via config arrays or config files.
MIT License
20 stars 3 forks source link

Service factories #59

Closed tomphp closed 8 years ago

tomphp commented 8 years ago

Resolves #43

tomphp commented 8 years ago

@localheinz there's some questionable naming in this PR - feel free to be picky ;-)

tomphp commented 8 years ago

@localheinz I've addressed the issues but it seems a forced push doesn't work with the new GitHub code review stuff - will bare that in mind in the future.

localheinz commented 8 years ago

@tomphp

Yes, it's a bit weird that the comments are still there, noticed it, too!