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

Code generator for haskell-servant framework #2009

Merged
merged 4 commits into from
Feb 1, 2016

Conversation

algas
Copy link
Contributor

@algas algas commented Jan 31, 2016

No description provided.

@wing328
Copy link
Contributor

wing328 commented Jan 31, 2016

@algas thanks for the PR! 🍻

One suggestion is to rename haskell to haskell-servant instead as developers may confuse haskell with the Haskell API client. (we'd the issue with nodejs in which developers expected a Nodejs API client but nodejs refers to Nodejs server generator instead)

@algas
Copy link
Contributor Author

algas commented Jan 31, 2016

@wing328 Thank you for your advice. I renamed it.


# if you've executed sbt assembly previously it will use that instead.
export JAVA_OPTS="${JAVA_OPTS} -XX:MaxPermSize=256M -Xmx1024M -DloggerPath=conf/log4j.properties"
ags="$@ generate -t modules/swagger-codegen/src/main/resources/haskell -i modules/swagger-codegen/src/test/resources/2_0/petstore.json -l haskell -o samples/server/petstore/haskell"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@algas should -l haskell be renamed to -l hashkell-servant and samples/server/petstore/haskell be renamed to samples/server/petstore/haskell-servant?

wing328 added a commit that referenced this pull request Feb 1, 2016
Code generator for haskell-servant framework
@wing328 wing328 merged commit 70cce74 into swagger-api:master Feb 1, 2016
@wing328
Copy link
Contributor

wing328 commented Feb 1, 2016

PR merged. Thanks again for the contribution.

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