Skip to content
This repository has been archived by the owner on Feb 25, 2019. It is now read-only.

Project merging? #1

Open
scottcorgan opened this issue Feb 9, 2016 · 2 comments
Open

Project merging? #1

scottcorgan opened this issue Feb 9, 2016 · 2 comments

Comments

@scottcorgan
Copy link

Great work. Saw this show up in my Github feed.

Looked similar to something I was doing a few years back, https://github.com/scottcorgan/injector, and I was wondering if I should just deprecate that and suggest this project?

Thoughts?

@christiansmith
Copy link
Member

@scottcorgan Wasn't aware of injector but it looks great! Dependency injection wasn't really the primary motivation for this, but it really helps to make it work overall. I'm not sure if anyone would want to use this type of plugin system if all they're looking for is DI. There's probably a little extra overhead – if not in terms of performance, at least in the programming boilerplate. Would love to do some mutual code review if you're down for that and see if we can find ways to improve both projects :)

@christiansmith
Copy link
Member

We have a writeup on the motivations for this project here, by the way.

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

No branches or pull requests

2 participants