-
Notifications
You must be signed in to change notification settings - Fork 822
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
Add base localisation by default #685
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
👋 After patching this version in I'm still getting the same warning. I made a change here: https://github.com/bryansum/XcodeGen/commit/f0d8eaab862b323e995fc5d63d22bd654cc510f8 which fixes it for me, although I'm not sure whether this is the right solution. |
If your fix works in your case, then I would think the warning is legitimate and would show up in a regular project. Xcode 11 wants you to have things localized to base |
Makes sense. In my case I have an |
Does the warning offer a fixit? See what it does |
yeah it adds "Base" to |
It doesn’t move your en files to base or copy them? If that’s all it does, then maybe we can indeed apply your fix |
No, it doesn't move them or copy them, it just adds that small change to add |
This adds a base localisation even if there are no Base localized files
Fixes #680