Skip to content
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

Fix bug (#251) #1319

Closed
wants to merge 5 commits into from
Closed

Fix bug (#251) #1319

wants to merge 5 commits into from

Conversation

birdmanmandbir
Copy link

Solve issue #251 and add corresponding test for it
Scope annotation in ProvidedBy handling should be added before create provided by binding

Solve issue google#251 and add corresponding test for it
Need consider scope annotation in ProvidedBy handling
copybara-service bot pushed a commit that referenced this pull request Apr 21, 2023
…ProvidedBy when using the @ProvidedBy as the provider.

Notably, this does *not* change things to allow scoping annotations on interfaces or abstract classes (even though theoretically we could, because the @ProvidedBy will be instantiating it). Scopes on abstract classes|interfaces will still throw an exception about the scope being misplaced.

Fixes #251 and fixes #1319.

PiperOrigin-RevId: 525525327
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants