[Erlang] Plant CI for server #5559
Conversation
lucafavatella
added
some commits
May 4, 2017
|
@lucafavatella thanks for the enhancement but the CI (travis) failed:
Please take a look when you've time. |
After I opened PR I realized that, so I need to first manage to install recent Erlang on Travis. Then I may go forward in |
lucafavatella
added
some commits
May 4, 2017
In df232af I bumped Erlang version. Corresponding CI build fails with a different reason as expected:
In c9c08c I performed a trivial change - a minor dep bump - but the corresponding CI build passes (!!!). The build shows no evidence of test having run. Shall I assume that somehow Travis got confused with some cache shared with the shortly afterwards committed 845e644 ? In 845e644 I disabled the erlang-server test in CI because known to fail, and it passes as expected. |
lucafavatella
changed the title from
[RFC][Erlang] Plant CI for server to [Erlang] Plant CI for server
May 5, 2017
I am unable to do this now. I suggest to review and merge PR as it is i.e. keeping CI still disabled, and then new PR (help warmly welcome, not sure whether and when I will be able to come back to this) may address compilation. |
It appears yes - as the same commit in my fork failed Travis CI as expected |
|
@lucafavatella agree with you that we can look into the CI (travis) issue later. |
wing328
merged commit ea16da8 into swagger-api:master May 6, 2017
wing328
added the
Enhancement: CI/Test
label
May 6, 2017
wing328
added this to the
v2.2.3
milestone
May 6, 2017
wing328
added the
Server: Erlang
label
May 6, 2017
lucafavatella
deleted the
lucafavatella:ci-erlang-server branch
May 10, 2017
davidgri
pushed a commit
to davidgri/swagger-codegen
that referenced
this pull request
May 11, 2017
|
|
lucafavatella + wing328 |
2adf1dd
|
lucafavatella commentedMay 4, 2017
This PR is not ready to be merged yet, and opened for receiving comments (RFC).
PR checklist
./bin/to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.shand./bin/security/{LANG}-petstore.shif updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)bin/erlang-petstore-server.sh./bin/security/)2.3.0branch for breaking (non-backward compatible) changes.Description of the PR
This PR plants CI for erlang-server. A related issue seems to be #3762
This PR tests that the sample compiles, that can be considered a very basic test.
I see the current version of the code generation leaves functions syntactically invalid - i.e. no body at all rather than e.g. raising a runtime error - hence the code will not compile. I suggest - and I plan to implement if I find time - the way forward is populating the (generated) body with
error(unimplemented)and the "correct" separator/terminator i.e.;or.. Any comments?