-
Notifications
You must be signed in to change notification settings - Fork 6k
Update samples after 2.2.2 #4901
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
Conversation
|
#4874 did the same for just the spring samples. |
|
The CI (travis) issue will be resolved by #4904. I'll let you know once it's merged into master. |
|
#4904 has been merged into master. Please rebase on the latest master to see if the CI (travis) issue goes away. |
984a292 to
593806a
Compare
|
Rebased. |
|
@wing328 now it complains about dependencies for Scalatra: https://travis-ci.org/swagger-api/swagger-codegen/jobs/207441746 |
|
Ah, I didn't actually regenerate the samples after rebasing. Trying it now for scalatra. |
|
Hmm, executing either |
|
I'll try to split this up into several PRs to isolate the problem. |
593806a to
73ace30
Compare
|
Build passed now. |
|
@ePaul thanks for the PR to update the samples. |
* Update petstore samples for Groovy. * Update samples for JaxRS. * Update samples for cpprest. * Update samples for dart. * Update samples for flash. * Update samples for go. * Update samples for html2. * Update samples for lumen. * Update samples for nancxfx. * Update samples for objc. * Update samples for sinatra. * Update samples for tizen. * Update samples for java-inflector.
PR checklist
./bin/to update Petstore sample so that CIs can verify the change.Description of the PR
No functional changes at all, just updated the samples for all the languages mentioned in #4898.
This will later allow to see what happened when fixing the bug there.