Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Using a Pimple 3 wrapper as an example of a Container-Interop container #228

Closed
wants to merge 1 commit into from

Conversation

Sam-Burns
Copy link

Using samburns/pimple3-containerinterop as example of DI container

@Sam-Burns
Copy link
Author

Ref. #129. \cc @weierophinney

@geerteltink
Copy link
Member

@weierophinney As explained in #129 I'm wondering if we should use another package that is extending Pimple in stead of placing it in another class. That way you won't loose all other functionality from Pimple and it's most likely easier to convert to a new Pimple release which supports container-interop.

@weierophinney
Copy link
Member

Closed in favor of #237. No hard feelings, @Sam-Burns — I just liked the approach of extending the pimple container better.

@Sam-Burns
Copy link
Author

@weierophinney it was an honour just to be nominated ;-)

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

Successfully merging this pull request may close these issues.

3 participants