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

Rename "guice" extension to "inject" #1406

Open
thboileau opened this issue Jan 21, 2024 · 2 comments
Open

Rename "guice" extension to "inject" #1406

thboileau opened this issue Jan 21, 2024 · 2 comments

Comments

@thboileau
Copy link
Contributor

cf this discussion: #1382

@kerbymart
Copy link

Hi @thboileau if the extension would be renamed to "inject" instead, does it mean the module should also support other DI?

e.g.
#1411

@jlouvel
Copy link
Member

jlouvel commented May 11, 2024

@kerbymart Based on our PoC for Dagger2, what would you like to see in a potential org.restlet.ext.inject extension? How could the code there be adapted to work equally well for Guice.

@Tembrel who had created the org.restlet.ext.guice extension initially is wondering if this deserves to stays an extension or if it shouldn't be a set of best practices. If that was the case, we could move the code to the official example and work on it from a documentation perspective.

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

No branches or pull requests

3 participants