glimmerjs / glimmer-di

MIT License
41 stars 9 forks source link

`factoryFor` injections #26

Closed pittst3r closed 7 years ago

pittst3r commented 7 years ago

This implements the factoryFor interface specified in https://github.com/emberjs/rfcs/blob/master/text/0150-factory-for.md and introduces concept of Creator (which is distinct from Factory). One registers Creators and retrieves Factories.

dgeb commented 7 years ago

Thanks @robbiepitts!

This LGTM. I think that tracking Creators separately from instances in the Registry (as described in #21) will pair nicely with this PR.