Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidate on en for development language #103

Merged
merged 1 commit into from
May 1, 2019

Conversation

bachand
Copy link
Collaborator

@bachand bachand commented May 1, 2019

I believe this is the root cause problem of #101.

Right now we are setting this to en-us in some places and Swift in other 馃う鈥嶁檪

In a standard Xcode project, this defaults to en. Apple suggested I set it to en.

In our production app, we are inconsistent. In one target we have it as English (which is invalid). In two others I checked we have it as en.

It feels inelegant setting a global variable, but this also feels like a step in the right direction of being more DRY.

Thank you Apple for your help, if you're watching 馃槃

I'm leaving that issue open until I rebase my other branch.

  • Built and ran in Xcode (make project)
  • Built and ran with Buck CLI (make debug)

cc: @qyang-nj @dfed

@bachand bachand requested review from dfed and qyang-nj May 1, 2019 06:25
Copy link
Collaborator

@qyang-nj qyang-nj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for cleaning this up!

@@ -53,7 +55,7 @@ def library_configs():
def app_binary_configs(name):
binary_specific_config = {
"ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES": "YES",
"DEVELOPMENT_LANGUAGE": "Swift",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

馃槺馃槺馃槺

Copy link
Collaborator

@dfed dfed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh no. Good sleuthing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants