[JaxRS-Spec] Fixes in generated pom #1035
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR checklist
./bin/
to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.sh
and./bin/security/{LANG}-petstore.sh
if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\
.master
,3.3.x
,4.0.x
. Default:master
.Description of the PR
Several improvements in the generated POM file for the
jaxrs-spec
generator:1/ Add
build-helper-maven-plugin
plugin in the generated pom to declaresrc/gen/java
as sources.=> Copied from the Jersey Jaxrs generator
The generator already output some content there:
Examples:
src/gen/java/
in jaxrs-specsrc/gen/java/
in jaxrs-spec-interfacesrc/gen/java/
in jaxrs-spec-interface-response2/ Jackson as dependency, because the models are using those annotations.
=> Copied from the Jersey Jaxrs generator
3/ Update
javax.ws.rs-api
to2.1.1
because the@PATCH
annotation (already used in the generated code) is not present in the2.0
version.Example Line 95:
openapi-generator/samples/server/petstore/jaxrs-spec/src/gen/java/org/openapitools/api/FakeApi.java
Lines 95 to 101 in 526e980