feat(desktop): do a check for triple dots instead of elipsis in i18n

[mgallien]

No description provided.

[rakekniven]

Looks like

docker-ci/translations/validateTranslationFiles.sh

Line 31 in 76a277f

if [ $(jq '.translations | keys[]' l10n/en_GB.json | grep '\.\.\.' | wc -l) -ne 0 ]; then

[rakekniven]

Fine for me. Thanks Matthieu

[nilsding]

LGTM, though I'd like to also add that check to our CI so we can catch it early on and not be surprised about the translations never being updated ;-)

https://github.com/nextcloud/desktop/blob/master/.github/workflows/check-translations.yml

[nickvergessen]

and not be surprised about the translations never being updated

Tobi and me receive error emails every night. But yeah if you can do it on CI that would be better.

[nilsding]

this file still needs to be added to the Dockerfile https://github.com/nextcloud/docker-ci/blob/master/translations-desktop/Dockerfile#L30

[mgallien]

closing this PR in favor of doing it with CI
for now only Qt managed translations are checked in CI
nextcloud/desktop#8522