Created by: jmini
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
,4.1.x
,5.0.x
. Default:master
. -
Copied the technical committee to review the pull request if your PR is targeting a particular programming language.
Description of the PR
Introduced with https://github.com/OpenAPITools/openapi-generator/pull/2388, newly generated generator created with CLI command:
openapi-generator meta -n myClientCodegen -t DOCUMENTATION -p com.my.company.example -o out
has a dependency on: org.openapitools:openapi-generator-cli
See the corresponding pom in the generated samples:
Input @jimschubert on Gitter
maybe we should revert OpenAPITools/openapi-generator#2388 and provide clearer instructions on how to debug manually?
The openapi-generator-cli
dependency is wrong, because the generators (normally in the org.openapitools:openapi-generator
project) do not have to know the CLI project org.openapitools:openapi-generator-cli
.
They can be used within the openapi-generator-maven-plugin
, the openapi-generator-gradle-plugin
or directly. In all those cases it is without the CLI.
As described in the readme of the generated plugin, the way to add the generator to the classpath of the CLI module is like this:
java -cp /path/to/openapi-generator-cli.jar:/path/to/your.jar org.openapitools.codegen.OpenAPIGenerator generate -g myClientCodegen -i /path/to/openapi.yaml -o ./test
The intention of the PR was to be able to create a test case: DebugCodegenLauncher
This is a great idea, but I think it would be much easier to create a MyclientcodegenGeneratorTest
class that call the MyclientcodegenGenerator
without the openapi-generator-cli
dependency.