Skip to content

Conversation

@jfiala
Copy link
Contributor

@jfiala jfiala commented Nov 6, 2016

PR checklist

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

Description of the PR

Language CXF server

  • add @Consumes/@produces annotations for JSON
  • add CLI-flag for switching between Spring XML or annotation-config
  • add CLI-flag for switching JAXB annotations in the model classes on/off
  • add a JbossFeature and CLI-flag to only generate jboss-specific files if requested
    (jboss-web.xml)

Language CXF client

  • add CLI-flag for switching JAXB annotations in the model classes on/off

Language jaxrs-resteasy:

  • add CLI flag for Jboss to language jaxrs-resteasy (with default true to be non-breaking)
  • added new test for CLI options JavaResteasyServerOptionsTest

#4088

walthema and others added 30 commits October 24, 2016 15:07
Added toString() to enums
Repair vendorExtensions due to broken merge.
Fallback to "src/main/java" for added source folder instead of a NullpointerException when no source folder ist configured in the pom
Default sourceFolder to src/main/java
Update pojo.mustache to repair broken merge
[JAXRS-CXF] Issue 4045 - Fixed enum marshalling & fromValue()
* Fixed deprecated usage -[JSONKeyMapper initWithDictionary:]

* Petstore regenerated

* Change minimal required version of JSONModel
Select application/json content-type in python generated client, if */* is in the list of content-types
DART: multi-word class names work properly now, fixes swagger-api#4117
[Java][JAX-RS] add vendor extension to handle void response in jaxrs-cxf
@jfiala
Copy link
Contributor Author

jfiala commented Nov 9, 2016

closed as PR has all changes from master after rebase.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants