-
-
Notifications
You must be signed in to change notification settings - Fork 7k
[Java] Retrofit2 Play! Framework 2.6.x support #901
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] Retrofit2 Play! Framework 2.6.x support #901
Conversation
@lukoyanov thanks for the PR. Some sample files were not up-to-date:
Please update them so that the CI build can pass. |
@lukoyanov can you please also update the CircleCI's pom under the CI folder to test https://github.com/OpenAPITools/openapi-generator/blob/master/CI/pom.xml.circleci#L901 |
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
@lukoyanov thanks for the PR, which has been included in the v3.2.3 release: https://twitter.com/oas_generator/status/1035200785066254336 |
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
Added Play! Framework 2.6.x support for retrofit2 clients
@bbdouglas (2017/07) @JFCote (2017/08) @sreeshas (2017/08) @jfiala (2017/08) @cbornet (2017/09) @jeff9finger (2018/01)