eclipse / microprofile-rest-client

MicroProfile Rest Client
Apache License 2.0
143 stars 72 forks source link

[308] Make CDI and MP Config imports optional #309

Closed andymc12 closed 3 years ago

andymc12 commented 3 years ago

Fixes #308

brevilo commented 3 years ago

Just curious, is there anything I can do to facilitate the merge?

andymc12 commented 3 years ago

Hi @brevilo - sorry for the delay - the weather was pretty nice here yesterday, so I took the day off. :) I wanted to make sure any other committer on the project had a chance to review. I'll go ahead and merge now. Thanks!

brevilo commented 3 years ago

No worries :) What's the release schedule to get this show-stopper out of the way for production?

andymc12 commented 3 years ago

I just released it as a release candidate for 2.0.1 - https://github.com/eclipse/microprofile-rest-client/releases/tag/2.0.1-RC1

In order to release this officially would require a lot of process and a lot of time (I think it's a minimum of two weeks). Unless there are other critical issues brought up, I suspect that we may not produce another official release until 2.1 (or 3.0) due to the process overhead. That said, the 2.0.1-RC1 release is exactly the same as the 2.0 release but with the changes added to make the CDI and MP Config package imports optional, so it should be fine for production usage. HTH

brevilo commented 3 years ago

Ok, understood. That should certainly be enough if the RC also shows up in maven central, for instance.

Nice one!