Add 'isFormStyle', 'isSpaceDelimited', and 'isPipeDelimited' flags to CodegenParameter#21240
Conversation
… CodegenParameter
thanks for the PR what about including these changes (cpp-qt templates) in this PR as well? cc @OpenAPITools/generator-core-team |
Just to clarify, we're not modifying existing templates, we're introducing a new generator. For example, I'm aiming to reduce the amount of generated code in the operation implementation, as mentioned in the issue description. |
|
👌 can you please add some tests in https://github.com/OpenAPITools/openapi-generator/blob/master/modules/openapi-generator/src/test/java/org/openapitools/codegen/DefaultCodegenTest.java to test these new flags to ensure these are working as expected moving forward? |
|
just merged it. i'll add some tests later with another pr thanks for your contribution |
This PR is related to issue #21239
These newly added flags lay the foundation for future changes in the generator's template files, particularly for the cpp-qt-client.
PR checklist
Commit all changed files.
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*.IMPORTANT: Do NOT purge/delete any folders/files (e.g. tests) when regenerating the samples as manually written tests may be removed.
master(upcoming7.x.0minor release - breaking changes with fallbacks),8.0.x(breaking changes without fallbacks)