Skip to content

swagger-codegen does not let me pick package name #627

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
josephmisiti opened this issue Apr 13, 2015 · 5 comments
Closed

swagger-codegen does not let me pick package name #627

josephmisiti opened this issue Apr 13, 2015 · 5 comments

Comments

@josephmisiti
Copy link

If I execute the following command in bash

    java -jar modules/swagger-codegen-cli/target/swagger-codegen-cli.jar generate  \
         -i http://developers.vestorly.com/v2/api-docs  \
         -l python   -o /Users/josephmisiti/mathandpencil/projects/vestorly/clients/v2/python

I see the following output:

writing file /Users/josephmisiti/mathandpencil/projects/vestorly/clients/v2/python/SwaggerPetstore-python/SwaggerPetstore/models/rss_feed.py
writing file /Users/josephmisiti/mathandpencil/projects/vestorly/clients/v2/python/SwaggerPetstore-python/SwaggerPetstore/models/post.py
writing file /Users/josephmisiti/mathandpencil/projects/vestorly/clients/v2/python/SwaggerPetstore-python/SwaggerPetstore/models/article.py
writing file /Users/josephmisiti/mathandpencil/projects/vestorly/clients/v2/python/SwaggerPetstore-python/SwaggerPetstore/models/event.py
writing file /Users/josephmisiti/mathandpencil/projects/vestorly/clients/v2/python/SwaggerPetstore-python/SwaggerPetstore/models/advisor.py
writing file /Users/josephmisiti/mathandpencil/projects/vestorly/clients/v2/python/SwaggerPetstore-python/SwaggerPetstore/models/source.py
writing file /Users/josephmisiti/mathandpencil/projects/vestorly/clients/v2/python/SwaggerPetstore-python/SwaggerPetstore/models/session.py

I do not want the folder SwaggerPetstore created on my machine

@fehguy
Copy link
Contributor

fehguy commented Apr 13, 2015

What's wrong with pets?

The generator needs this:

#616

Then you will be able to configure it. Essentially it means either passing in a JSON configuration object or a bunch of args. Still deciding which way to take that.

@josephmisiti
Copy link
Author

I just mean I am using swagger-codegen for a public facing API and SwaggerPetstore has nothing to do with. If I understand correctly, does this mean that the functionality doesnt currently exist per #616

(I am working off of develop_2.0)

@fehguy
Copy link
Contributor

fehguy commented Apr 13, 2015

There are two options.

  1. Everything in the codegen is intended to be modified. Typically people will do so by extending the functionality in the language classes.

  2. We can choose different defaults in the codegen project to get the pets out. I agree it should be SwaggerClient instead of Petstore.

@josephmisiti
Copy link
Author

cool thank you!

@webron
Copy link
Contributor

webron commented May 4, 2015

Closing this issue as it is indeed covered by #616 (though not yet resolved). Please add any relevant comments there.

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

No branches or pull requests

3 participants