-
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
[Swift3] add en_US_POSIX locale to date formatters #5602
Conversation
@julienfouilhe thanks for the PR. |
👍 possible / easy to isolate a unit test in the example suite? |
@jgavris Where are they located? I don't see any swift tests. |
@julienfouilhe you can find the test cases for Swift3 Petstore client under https://github.com/swagger-api/swagger-codegen/tree/master/samples/client/petstore/swift3 |
@wing328 @jgavris After giving it some thought I'm not sure how to test this as it depends on the user's device locale. But here is a link to Apple's recommendations: https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/DataFormatting/Articles/dfDateFormatting10_4.html
|
I've had success swizzling |
i'm ok w/ that @wing328 |
👍 as well, sorry for the delay |
PR checklist
./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)2.3.0
branch for breaking (non-backward compatible) changes.Description of the PR
The same thing has been done for the swift generator, it resolves the same issue, described there: #2159
#5008
The only file edited is this one:
modules/swagger-codegen/src/main/resources/swift3/Models.mustache