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

Add support for interfaces #5

Open
hkakutalua opened this issue Sep 11, 2019 · 2 comments
Open

Add support for interfaces #5

hkakutalua opened this issue Sep 11, 2019 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@hkakutalua
Copy link

hkakutalua commented Sep 11, 2019

Is not possible to instantiate concrete classes from their base classes, this a severe limitation.

BaseClass base = get<BaseClass>(); // Instantiate ConcreteClass

The code below would probably be an API breaking change:

factory<CacheStore>(() => FileCacheStore(inject()))
@hkakutalua hkakutalua self-assigned this Sep 11, 2019
@hkakutalua hkakutalua added the enhancement New feature or request label Sep 11, 2019
@PDDStudio
Copy link

Hello.
I was about to use this library.
Unfortunately, I realized during debugging that interface/abstract class injection is not supported.
W/o being able to resolve base classes from given implementations this library is useless for me.

Is there any progress on this issue already? Or is it better to check for a working alternative?

@hkakutalua
Copy link
Author

Hi @PDDStudio, thank you for bringing this topic to our attention as it was forgotten due to other priority tasks inside our company.

We'll see implement if possible in this week, but you're also welcome to contribute with a pull request.

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants