-
Notifications
You must be signed in to change notification settings - Fork 6k
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
ApiClient Bugfix: containsString removed to support iOS 7 #2210
Conversation
@svenluzar thanks for the PR. Please update the ObjC Petstore sample and also run the test locally to ensure all the test cases pass:
Ref: https://github.com/swagger-api/swagger-codegen/blob/master/CONTRIBUTING.md |
Thanks for your hint. What's the exact problem with my pull request? The tests already have executed with success. ** TEST SUCCEEDED ** 16:57:50.373 xctest[14414:1293055] _XCT_testBundleReadyWithProtocolVersion:minimumVersion: reply received |
Sorry I was not clear. There's no problem with your PR. As part of the change process, we want the contributor to update the Petstore sample so that CI (travis) can test the change or the contributor can also verify the change themselves locally. PR merged. Thanks for the contribution. |
ApiClient Bugfix: containsString removed to support iOS 7
No description provided.