-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
[Java][Client] Add CLI option to select OpenAPI annotation library #8801
[Java][Client] Add CLI option to select OpenAPI annotation library #8801
Conversation
CI failure is from hitting Docker Hub pull limits. |
Hi what is the status? Get rid o the swagger annotations would be grate. |
@wing328 Would you be able to take a look at this? |
@djnalluri thanks for the PR and sorry for the delay in reviewing the change. Do we need to update the following pojo.mustache files as well?
|
I just updated the |
# Conflicts: # modules/openapi-generator/src/main/resources/Java/libraries/jersey2/pojo.mustache # modules/openapi-generator/src/main/resources/Java/libraries/native/pojo.mustache # modules/openapi-generator/src/main/resources/Java/pojo.mustache
# Conflicts: # modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/JavaClientCodegen.java
@wing328 Would you be able to take another look at this? |
I can see that this "issue" drives a lot of people and that there are multiple issues that try to provide some means of removing the annotations. (4270, 5058, 7612) Can this PR be accepted ? I guess it will make a lot of people happy. |
This PR doesn't disable all usages of swagger-annotations in 'java' generator templates, opened a more complete one |
That PR(#10581) doesn't provide room for alternate libraries but does get the job done. I glanced over the PR and it removes swagger annotations being added as dependencies in the supporting files(POM's, etc.). Supporting files aren't covered at all in this PR. If the other PR is easier to get approved, I'm all for it. Considering the upcoming breaking 6.0 release for December, I can patch this PR to take those changes into account and re-add support for alternate libraries. |
hi @djnalluri I hope that the atomicity, completeness and simplicity of changes could make the review process easier (but of course we also can cherrypick my commits to your PR). @wing328 hi, could you tell us what way is preferable for speeding up review process? |
Implemented in #12422. |
Closing as an alternative annotation name has been implemented in #13869. |
A new CLI option named "oaiAnnotationLib" can be used to select the OpenAPI annotation library used to annotate models. The current options are "none" and "swagger2". Placeholder sections have been left in the code and templates for future addition of a Microprofile option. These changes are a subset from #8716.
Fixes #8719.
PR checklist
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*
.For Windows users, please run the script in Git BASH.
master
,5.1.x
,6.0.x
Technical Committee:
@bbdouglas (2017/07) @sreeshas (2017/08) @jfiala (2017/08) @lukoyanov (2017/09) @cbornet (2017/09) @jeff9finger (2018/01) @karismann (2019/03) @Zomzog (2019/04) @lwlee2608 (2019/10) @nmuesch (2021/01)