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

Allow for having CDI on pure Jersey Client without Jersey Server #4695

Merged
merged 2 commits into from
Jan 26, 2021

Conversation

jansupol
Copy link
Contributor

Signed-off-by: jansupol jan.supol@oracle.com

… dependencies

Signed-off-by: jansupol <jan.supol@oracle.com>
*/
public static void bindProviders(ComponentBag componentBag,
RuntimeType constrainedTo,
Set<Class<?>> registeredClasses,
InjectionManager injectionManager) {
InjectionManager injectionManager,
Collection<ComponentProvider> componentProviders) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As this method is public it is possible that it is already referenced by user code. Shouldn't we duplicate this method with previous arguments?.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The general contract is that an internal package can be a subject of change. But yes, adding an extra method won't hurt.

Signed-off-by: jansupol <jan.supol@oracle.com>
@senivam senivam merged commit 7bf9e5d into eclipse-ee4j:master Jan 26, 2021
@jansupol jansupol self-assigned this Jan 26, 2021
@jansupol jansupol added the cdi label Jan 26, 2021
@jansupol jansupol added this to the 2.34 milestone Jan 26, 2021
@jansupol jansupol deleted the cdiOnClient branch January 26, 2021 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants