William Cheng
William Cheng
> Build with samples profile (-Psamples) fails on server/java-msf4j, and this is not related to the fix. I've fixed that in the latest master.
@SergeyLyakhov can you merge the latest `origin/master` to see if the CI errors are fixed?
@SergeyLyakhov thanks. Let's see if all the tests pass.
@SergeyLyakhov I would appreciate if you can do (1) and (2) in a single PR based on the latest master. I'll do (3) and (4) later after merging the PR.
@ignaciomolina let me review tomorrow. Sorry for the delay.
@ignaciomolina the change breaks the Pistache generator: ``` [main] INFO io.swagger.parser.Swagger20Parser - reading from modules/swagger-codegen/src/test/resources/2_0/petstore.yaml [main] INFO io.swagger.codegen.AbstractGenerator - writing file /root/src/github.com/swagger-api/swagger-codegen/samples/server/petstore/pistache-server/model/ApiResponse.h [main] INFO io.swagger.codegen.AbstractGenerator - writing file /root/src/github.com/swagger-api/swagger-codegen/samples/server/petstore/pistache-server/model/ApiResponse.cpp [main]...
I was able to repeat the issue locally by running `./bin/pistache-server-petstore.sh`.
@ignaciomolina can you try https://app.shippable.com/download/jobConsoles?jobId=5a965e1b9e550508008cef87? You will need to register an account at shippable and log in to obtain a valid token.
Thanks. I will review tonight.
cc the following technical committees as the change affects multiple generators: - Ada: @stcarrez @micheleISEP - Apex: @asnelling - C++: @ravinikam @stkrwork @fvarose - Pistache: @sebymiano