-
Notifications
You must be signed in to change notification settings - Fork 62
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
Refactor tests and QuarkusCxfProcessor #499
Conversation
and also in native mode * Put Woodstox stuff into a separate module so that Camel Quarkus can reuse it * Put jakarta.xml.ws related build steps into a separate WsApiProcessor so that there is some structure in the processors
…parate SaajImplProcessor class
…entations already available in io.quarkiverse.cxf.deployment.WsApiProcessor.registerForReflection()
…stead of abstract classes, fix related issues
…he dedicated method
c5b4a37
to
d5741aa
Compare
Bumps [woodstox-core](https://github.com/FasterXML/woodstox) from 6.2.7 to 6.3.1. - [Release notes](https://github.com/FasterXML/woodstox/releases) - [Commits](FasterXML/woodstox@woodstox-core-6.2.7...woodstox-core-6.3.1) --- updated-dependencies: - dependency-name: com.fasterxml.woodstox:woodstox-core dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
bb15063
to
fc57f93
Compare
Would anybody like to review? |
I can review in a couple hours. |
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.
LGTM @ppalaga ... one minor comment is that CXF 3.5.3 depends on woodstox-core
6.2.8. I'm not sure if switching that to 6.3.1 may have any unintended side effects?
Woodstox folks care for their backwards compatibility. Citing from their release page
So going to a newer minor should not be a problem. Also, if our tests pass it should be fine. |
No description provided.