Skip to content

[Elixir] Add elixir client#4675

Merged
wing328 merged 5 commits intoswagger-api:masterfrom
niku:add-elixir-client
Jan 30, 2017
Merged

[Elixir] Add elixir client#4675
wing328 merged 5 commits intoswagger-api:masterfrom
niku:add-elixir-client

Conversation

@niku
Copy link
Copy Markdown
Contributor

@niku niku commented Jan 30, 2017

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

Add elixir client(alpha)

Related https://github.com/niku/swagger-codegen-for-elixirclient/issues/3

  • I think this client generator is alpha version, how to notify this status?
  • How to write ./bin/security/...-petstore.sh ?

* @return A string value for the help message
*/
public String getHelp() {
return "Generates an elixir client library.";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@niku I suggest Generates an elixir client library (alpha). to indicate the API lib is still alpha.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you.
I have rewrited and force pushed.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@niku thanks. I'll merge it into master shortly.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

How to write ./bin/security/...-petstore.sh

I'll take care of it. No worry :)

@niku niku force-pushed the add-elixir-client branch from 68b140f to 001b1c4 Compare January 30, 2017 09:12
@wing328 wing328 added this to the v2.2.2 milestone Jan 30, 2017
@wing328 wing328 merged commit f77bee8 into swagger-api:master Jan 30, 2017
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jan 30, 2017

@niku niku deleted the add-elixir-client branch January 30, 2017 13:05
@wing328 wing328 changed the title [WIP] Add elixir client [Elixir] Add elixir client Feb 20, 2017
davidgri pushed a commit to davidgri/swagger-codegen that referenced this pull request May 11, 2017
* Add elixir client

* Add test for elixir client

* Add shell script for generating sample codes for elixir client

It just copied from bin/bash-petstore.sh

* Make elixir-petstore.sh to generate sample codes for elixir client

* Add sample codes for elixir client
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