Pull iOS stringdicts with default source (part 2)

[mpivchev]

Follow up of #956

[nickvergessen]

1. The commit message is not acceptable
2. This PR breaks the sync because there are no arguments -force and so on, they need 2 dashes
   • Fix amended in chore: Use 'sourceastranslation' for talk-ios plurals #983
3. The image was not rebuilt after this was merged, so the above breakage did not yet get deployed…

[mpivchev]

@nickvergessen

1. You are correct, will be resolved next time
2. Are you sure? I tested locally and -force is recognized:

3. It was deployed. We tested via forcing transifex sync and it completed successfully.
   Still works to this day:

[nickvergessen]

At least per documentation:

root@50c69e62f679:/# tx pull --help
NAME:
   tx pull - tx pull [options] [resource_id...]

USAGE:
   tx pull [command options] [arguments...]

OPTIONS:
   --xliff                             Download translation files in xliff format (default: false)
   --json                              Download translation files in json format (default: false)
   --content_encoding value, -e value  The encoding of the file. This can be one of the following:
    'text', 'base64' (default: "text")
   --mode value, -m value  The translation mode of the downloaded file. This can be one of the following:
    'default', 'reviewed', 'proofread', 'translator', 'untranslated',
    'onlytranslated', 'onlyreviewed', 'onlyproofread', 'sourceastranslation' (default: "default")
   --force, -f                  Force the download of the translationsfiles regardless of whether timestamps on the local computer are newer than those on the server (default: false)
   --languages value, -l value  Download specific languages, comma separated Transifex language codes
   --source, -s                 Download source file only (default: false)
   --translations, -t           Downloads translations files (default) (default: false)
   --disable-overwrite, -d      Whether skip existing files (default: false)
   --keep-new-files             Used with --disable-ovewrite to create new files if file already exists with a '.new' extension. (default: false)
   --skip                       Whether to skip on errors (default: false)
   --use-git-timestamps         Compare local files to their Transifex version by their latest commit timestamps. Use this option, for example, when cloning a Git repository. (default: false)
   --branch value               Push to specific branch (use empty argument '' to use the current branch, if it can be determined) (default: "-1")
   --all, -a                    Whether to download all files (default: false)
   --resources value, -r value  Backwards compatibility with old client to fetch resource ids
   --minimum-perc value         Specify the minimum acceptable percentage of a translation mode in order to download it. (default: -1)
   --workers value, -w value    How many parallel workers to use (max 20) (default: 5)
   --silent                     Whether to reduce verbosity of the output (default: false)
   --pseudo                     Generate mock string translations (default: false)
   --help, -h                   show help (default: false)

But anyway it's fixed now

[mpivchev]

At least per documentation:

root@50c69e62f679:/# tx pull --help
NAME:
   tx pull - tx pull [options] [resource_id...]

USAGE:
   tx pull [command options] [arguments...]

OPTIONS:
   --xliff                             Download translation files in xliff format (default: false)
   --json                              Download translation files in json format (default: false)
   --content_encoding value, -e value  The encoding of the file. This can be one of the following:
    'text', 'base64' (default: "text")
   --mode value, -m value  The translation mode of the downloaded file. This can be one of the following:
    'default', 'reviewed', 'proofread', 'translator', 'untranslated',
    'onlytranslated', 'onlyreviewed', 'onlyproofread', 'sourceastranslation' (default: "default")
   --force, -f                  Force the download of the translationsfiles regardless of whether timestamps on the local computer are newer than those on the server (default: false)
   --languages value, -l value  Download specific languages, comma separated Transifex language codes
   --source, -s                 Download source file only (default: false)
   --translations, -t           Downloads translations files (default) (default: false)
   --disable-overwrite, -d      Whether skip existing files (default: false)
   --keep-new-files             Used with --disable-ovewrite to create new files if file already exists with a '.new' extension. (default: false)
   --skip                       Whether to skip on errors (default: false)
   --use-git-timestamps         Compare local files to their Transifex version by their latest commit timestamps. Use this option, for example, when cloning a Git repository. (default: false)
   --branch value               Push to specific branch (use empty argument '' to use the current branch, if it can be determined) (default: "-1")
   --all, -a                    Whether to download all files (default: false)
   --resources value, -r value  Backwards compatibility with old client to fetch resource ids
   --minimum-perc value         Specify the minimum acceptable percentage of a translation mode in order to download it. (default: -1)
   --workers value, -w value    How many parallel workers to use (max 20) (default: 5)
   --silent                     Whether to reduce verbosity of the output (default: false)
   --pseudo                     Generate mock string translations (default: false)
   --help, -h                   show help (default: false)

But anyway it's fixed now

Yeah i saw it too. But I guess single dashes are also allowed.

[SystemKeeper]

I assume it’s then recognized as the shorthand version (-f /-a) and the rest is just skipped, so it works by chance.