Skip to content

Conversation

inidona
Copy link
Contributor

@inidona inidona commented Apr 22, 2018

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: 3.0.0 branch for changes related to OpenAPI spec 3.0. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

(details of the change, additional tests that have been done, reference to the issue for tracking, etc)
#8055 creates sources.jar and javadoc jar from gradlebuild for deploying to jfrog public snapshot repository

@inidona inidona changed the title Issue 8055 [codegen][java]Issue 8055: create javadoc and source jar from gradle build Apr 22, 2018
@inidona
Copy link
Contributor Author

inidona commented Apr 22, 2018

@bbdouglas (2017/07) @JFCote (2017/08) @sreeshas (2017/08) @jfiala (2017/08) @lukoyanov (2017/09) @cbornet (2017/09) @jeff9finger (2018/01) please review

Copy link
Contributor

@JFCote JFCote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

}



Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why we need 3 empty lines here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are right, i removed 2 lines now because of the other formattings in the file

@jeff9finger
Copy link

LGTM 👍

@inidona
Copy link
Contributor Author

inidona commented Apr 26, 2018

@wing328 is it possible to merge ?

@inidona
Copy link
Contributor Author

inidona commented May 1, 2018

@wing328 ping

@inidona
Copy link
Contributor Author

inidona commented May 9, 2018

@wing328 whats wrong with this pr ?

@wing328 wing328 added this to the v2.4.0 milestone May 10, 2018
@wing328 wing328 merged commit cdffe27 into swagger-api:master May 10, 2018
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.

4 participants