Skip to content

@ProvidedBy requires a Guice provider #808

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
gissuebot opened this issue Jul 7, 2014 · 0 comments · Fixed by #926
Closed

@ProvidedBy requires a Guice provider #808

gissuebot opened this issue Jul 7, 2014 · 0 comments · Fixed by #926
Labels
imported

Comments

@gissuebot
Copy link

From pier@usrz.com on May 25, 2014 03:20:53

Can @ProvidedBy require a javax.inject.Provider instead of a com.google.inject one?

Given that the latter extends the former, all code should work (theoretically)...

Attachment: gist
   ProvidedBy.java

Original issue: http://code.google.com/p/google-guice/issues/detail?id=808
@tavianator tavianator mentioned this issue May 7, 2015
Merged
@sameb sameb closed this as completed in #926 May 7, 2015
ShoOgino pushed a commit to ShoOgino/guiceFile that referenced this issue Oct 14, 2020
@tavianator
Make @ProvidedBy take a javax.inject.Provider.
f39b444 
Fixes google/guice#808.
ShoOgino pushed a commit to ShoOgino/guiceMethod that referenced this issue Oct 14, 2020
@tavianator
Make @ProvidedBy take a javax.inject.Provider.
8850804 
Fixes google/guice#808.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
imported
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make @ProvidedBy take a javax.inject.Provider.
1 participant
@gissuebot