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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Script improvement #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Script improvement #2

wants to merge 1 commit into from

Conversation

jifang
Copy link

@jifang jifang commented Feb 2, 2016

  1. Update the python script for generating the string file from swift source.
    Now, it supports generating string from multiple inputs. The usage is
    similar to the Apple's genstrings tool.
    Usage: find /path/to/source -iname "*.swift" | xargs python ./genstrings_swift.py -o ./App/en.lproj/ ./App/Source/LocalizationConstants.swift
  2. Fix the bug in python script for merging the string file. When the string
    value contains escaped double quotes, it will be excluded from the string
    file. For example, following lines will be striped from the final output
    /* String with quotes */
    "EscapedQuotes" = ""String with quotes"";

1. Update the python script for generating the string file from swift source.
   Now, it supports generating string from multiple inputs. The usage is
   similar to the Apple's genstrings tool.
   Usage: find /path/to/source -iname "*.swift" | xargs python ./genstrings_swift.py -o ./App/en.lproj/ ./App/Source/LocalizationConstants.swift
2. Fix the bug in python script for merging the string file. When the string
   value contains escaped double quotes, it will be excluded from the string
   file. For example, following lines will be striped from the final output
   /* String with quotes */
   "EscapedQuotes" = "\"String with quotes\"";
@philippb
Copy link
Contributor

philippb commented Feb 2, 2016

Thank you. We try to look into it this week.

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.

3 participants