Skip to content

Conversation

@daveish
Copy link
Contributor

@daveish daveish commented Aug 25, 2017

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./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\.
  • Filed the PR against the correct branch: master for non-breaking changes and 3.0.0 branch for breaking (non-backward compatible) changes.

Description of the PR

Fixes a bug where response annotations for JaxRS server codegen got duplicated. See #2588 for detailed bug description.

@daveish
Copy link
Contributor Author

daveish commented Aug 25, 2017

circleci failed due to changes in the samples which came from the master branch. I'm looking into it.

@daveish
Copy link
Contributor Author

daveish commented Aug 25, 2017

The sample generator looked for jersey1 template files in "JavaJaxRS/libraries/jersey1/libraries/jersey1" instead of "JavaJaxRS/libraries/jersey1".

@daveish
Copy link
Contributor Author

daveish commented Aug 25, 2017

There is another commit on the master, which breaks the Java codegen functionality. Im working on it.

@wing328
Copy link
Contributor

wing328 commented Aug 29, 2017

I've performed a few tests and the result is good. Thanks for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants