-
Notifications
You must be signed in to change notification settings - Fork 888
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
RESTEASY-2194 fixing CDI property injector to always work on contextu… #1942
Conversation
resteasy-cdi/pom.xml
Outdated
<dependency> | ||
<groupId>org.jboss.weld</groupId> | ||
<artifactId>weld-api</artifactId> | ||
<version>3.1.Final</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the version element here and let the dependencies bom control it
c890a0e
to
c2a7736
Compare
Done. Thanks for review. |
@istudens thanks; please note this also needs to be rebased against master now |
@asoldano It should be. I did rebase it against master and the other one against 3.7. |
…al instance instead of client proxy
c2a7736
to
795e297
Compare
I've just rebased both PRs again. |
Thanks @istudens , I've merged the PR. That said, any chance you can also provide an integration test for this? |
@asoldano Sure, I have a testcase in EAP testsuite, but I am working on a testcase in RestEasy directly. I will get back to you. |
…al instance instead of client proxy
https://issues.jboss.org/browse/RESTEASY-2194