Skip to content
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

[Groovy] improve code parsing body/form params #2879

Conversation

karismann
Copy link
Contributor

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, ./bin/openapi3/{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\. If contributing template-only or documentation-only changes which will change sample output, be sure to build the project first.
  • Filed the PR against the correct branch: master, 3.4.x, 4.0.x. Default: master.

Description of the PR

  • improve code parsing body/form params
  • add removeOption for CliOptions in DefaultCodeGen class
  • remove overrided method identical to parent (AbstractJavaCodeGen and AbstractApexCodeGen)

@karismann karismann force-pushed the Groovy-improve-code-parsing-body-form-params branch from c583830 to 275f9e9 Compare May 14, 2019 12:54
@wing328 wing328 added this to the 4.0.1 milestone May 15, 2019
@karismann karismann force-pushed the Groovy-improve-code-parsing-body-form-params branch from d70517f to 6551a08 Compare May 17, 2019 08:24
@wing328 wing328 merged commit 6bffe4d into OpenAPITools:master May 20, 2019
@karismann karismann deleted the Groovy-improve-code-parsing-body-form-params branch May 20, 2019 06:10
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