-
Notifications
You must be signed in to change notification settings - Fork 6k
New sync master to 2.3.0 #5055
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
Closed
ePaul
wants to merge
165
commits into
swagger-api:sync_master_230
from
ePaul:new-sync-master-to-2.3.0
Closed
New sync master to 2.3.0 #5055
ePaul
wants to merge
165
commits into
swagger-api:sync_master_230
from
ePaul:new-sync-master-to-2.3.0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ignore dist folder
…api#4695) If the swagger model contains a class named 'Object' some mustache templates will generate non compiling **Java** code.
…to boolean. (swagger-api#4747) CodegenOperation and CodegenProperty classes have boolean fields instead of Boolean. This fix changes Boolean fields of CodegenParameter and CodegenResponse to boolean as well. Boolean fields are not necessary. Other classes interacting with them have to check for null before interacting with them which is unnecessary and leads to ugly code.
* add boolean datatype flags to CodegenResponse and logic to set them in DefaultCodegen * use Boolean.TRUE.equals to check for isResponseBinary or isResponseFile
While creating a static library from the generated files, compilation issues arose due to QDateTime only being forward declared. Adding the include resolves the issue.
…-api#4712) * add language for resteasy eap swagger-api#4512 * first implementation of jaxrs-resteasy-eap swagger-api#4512 * add support for joda and java8 datetime types swagger-api#4512 * add new file JacksonConfig and new sample eap-joda swagger-api#4512 * add dynamic swagger support to jaxrs-resteasy-eap swagger-api#4512 * adapt readme for eap swagger-api#4512 * add tests for jaxrs-resteasy-eap swagger-api#4512 * cleanup tabs swagger-api#4512 * cleanup tabs swagger-api#4512 * cleanup tabs swagger-api#4512 * cleanup tabs / regenerate without joda swagger-api#4512 * regenerate with updated templates swagger-api#4512 * fix test (wrong setter invoked) swagger-api#4512 * fix test swagger-api#4512 * fix no invocation swagger-api#4512 * replace tabs with spaces swagger-api#4512
updated parser version
…-api#4912) * Run mvn in batch-mode to be less verbose on download * Build and upload Swagger Codegen CLI Docker image on release * Ensure we only push master and tags to Docker Hub
…r-codegen into jdupouy-2-call-visibility2
* Fix (partially) swagger-api#4898 for groovy. This also adds some TODOs for the missing path, form and body parameters. * Update Groovy samples (after partial fix for swagger-api#4898)
(except clojure + scalatra, because those are broken, will fixed with later master merge.)
0b0b970 to
9be178b
Compare
Contributor
Author
|
This broke on Travis because https://nexus.codehaus.org/snapshots/ is not reachable. AppVeyor has the same problem as for #5056 and #5054: swagger-parser:jar:1.0.26-SNAPSHOT is not in sonatype-snapshots anymore. |
Contributor
Author
|
Replaced by #5079. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
❌ Do not merge yet, this is only here for CI to run over it. ❌
PR checklist
./bin/to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.shand./bin/security/{LANG}-petstore.shif updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)2.3.0branch for breaking (non-backward compatible) changes.Description of the PR
After @wing328 got his sync_master_230 branch (which merged master and 2.3.0 of 2017-02-09) to work, this tries to merge a newer master version (from today) into the result. This is not meant to be merged now, first sync_master_230 should be merged into 2.3.0.