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 multibind class provider #197

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Letsch22
Copy link

@Letsch22 Letsch22 commented May 27, 2022

Adds the MultiBindClassProvider which provides instances of classes from a list of class types when using the multibind API.

Resolves #45

@Letsch22 Letsch22 force-pushed the multibind-class-provider branch from d0d6f42 to b04ab6f Compare June 3, 2022 21:05
@Letsch22 Letsch22 force-pushed the multibind-class-provider branch from b04ab6f to 0934458 Compare June 4, 2022 08:16
@jstasiak
Copy link
Collaborator

Thank you for this, I'll have to think about this interface

@KrzysztofMadejski
Copy link

I just ran into the same need. It's quite of a gap that multibind can't work with ClassProvider as normal bind would.

@tFerrellBastian
Copy link

Just curious as to the status of this pr... The issue is closed, but the PR has not been merged in and I just ran into this issue in my code. Is there something I can do to help close out this pr?

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

Successfully merging this pull request may close these issues.

Multibinder requires instances.
4 participants