[GitHub] [cxf] andymc12 commented on pull request #805: Make CDI and MP Config dependencies optional for MicroProfile REST client

2021-06-04 Thread GitBox
andymc12 commented on pull request #805: URL: https://github.com/apache/cxf/pull/805#issuecomment-853417886 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries

[GitHub] [cxf] andymc12 commented on pull request #805: Make CDI and MP Config dependencies optional for MicroProfile REST client

2021-06-03 Thread GitBox
andymc12 commented on pull request #805: URL: https://github.com/apache/cxf/pull/805#issuecomment-853890195 @rmannibucau Thanks for catching that. I have a fix in PR #810 - would you mind reviewing? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [cxf] andymc12 commented on pull request #805: Make CDI and MP Config dependencies optional for MicroProfile REST client

2021-06-02 Thread GitBox
andymc12 commented on pull request #805: URL: https://github.com/apache/cxf/pull/805#issuecomment-853417886 @brevilo @rmannibucau I just merged #808 which tests that the CXF MP Rest Client works even when no CDI or MP Config is present on the classpath. That shows that it should work at

[GitHub] [cxf] andymc12 commented on pull request #805: Make CDI and MP Config dependencies optional for MicroProfile REST client

2021-05-30 Thread GitBox
andymc12 commented on pull request #805: URL: https://github.com/apache/cxf/pull/805#issuecomment-851036627 Hi @brevilo - I'm on holiday today and tomorrow, but I can look at adding a test case when I get back. Thanks! On Sun, May 30, 2021 at 10:56 AM brevilo ***@***.***> wrote:

[GitHub] [cxf] andymc12 commented on pull request #805: Make CDI and MP Config dependencies optional for MicroProfile REST client

2021-05-28 Thread GitBox
andymc12 commented on pull request #805: URL: https://github.com/apache/cxf/pull/805#issuecomment-850610993 @rmannibucau fwiw, the ConfigFacade works in Open Liberty with and without MP Config. That said, it should be possible to create a new test project that runs a small portion