Package names in the finch generator should not contain petstore

[clasnake]

I found that in FinchServerCodegen, the packageName and invokerPackage are set to "io.swagger.petstore...".
petstore is just the sample and I guess the package names here should not contain "petstore".

[clasnake]

@wing328 @jimschubert Just had a quick look at the finch generator and found the above issue.

[wing328]

@clasnake have you tried customizing via the options?

swagger-codegen|master⚡ ⇒ java -jar modules/swagger-codegen-cli/target/swagger-codegen-cli.jar config-help -l finch  

CONFIG OPTIONS
	packageName
	    Finch package name (e.g. io.swagger.petstore). (Default: io.swagger.petstore)

	modelPackage
	    package for generated models

	apiPackage
	    package for generated api classes

[clasnake]

@wing328 I guess even the default should not contain petstore. Can be "io.swagger"

[wing328]

@clasnake you're definitely right. What default would you suggest?

https://github.com/swagger-api/swagger-codegen/blob/master/modules/swagger-codegen/src/main/java/io/swagger/codegen/languages/FinchServerCodegen.java#L18

https://github.com/swagger-api/swagger-codegen/blob/master/modules/swagger-codegen/src/main/java/io/swagger/codegen/languages/FinchServerCodegen.java#L23

[clasnake]

@wing328 How about "io.swagger"?

[wing328]

@clasnake sounds better to me as it doesn't contain "petstore" 😄

Please feel free to submit the PR with updated petstore sample for Finch.

[wing328]

Closed via #4755