-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Fixes for JAXRS-CXF Codegen #2256
Conversation
@hiveship your first commit https://github.com/hiveship/swagger-codegen/commit/37e62b89bc7f4b1be2f7efeaf70f1f42dd7ea652 is not linked to your github account and therefore that won't count as your contribution in https://github.com/swagger-api/swagger-codegen/graphs/contributors. |
``` | ||
|
||
This is no longer necessary if you are using CXF >=v3.x | ||
|
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.
I found some grammatical mistakes and make an update in another commit.
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.
OK thank you. I also updated the file to remove my company name in the 'CXFInterfaceComparator' package. Pease, include it.
Got an error testing the PR:
Please update the Petstore samples for |
I've take a look this mornig and I don't see why this bug occurs (CXFCodegen, JAXRSCodegen & Java Codegen don't import 'enumClass.mustache'). I'm currently using this version on some of my projects (that use enums class and I don't have any problem. |
Updated. |
@hiveship please take your time. Let me know after you've updated it and thenI'll give it another try. |
@wing328 Could you please retry ? I just added a commit and the sample generation is OK for me. |
i'm the original author of the CXF Codegen. I've made some updates. I'm now using this codegen in a project, it's OK.