Merge pull request #69255 from BrettDong/discard_invalid_po

Discard invalid PO files in pull translations GHA workflow

.github/workflows/pull-translations.yml

@@ -39,6 +39,8 @@ jobs:
         env:
           TX_TOKEN: ${{ secrets.TX_TOKEN }}
         run: tx pull --force
+      - name: "Discard invalid translations"
+        run: ./lang/discard_invalid_po.sh
       - name: "Update stats"
         run: ./lang/update_stats.sh
       - name: Create Pull Request

lang/discard_invalid_po.sh

@@ -0,0 +1,19 @@
+#!/usr/bin/env bash
+
+# Sometimes PO files pulled from Transifex are not accepted by GNU gettext, for example
+#   lang/po/hu.po:430257: 'msgid' and 'msgstr' entries do not both begin with '\n'
+#   lang/po/hu.po:534682: 'msgid' and 'msgstr' entries do not both end with '\n'
+#   lang/po/hu.po:534692: 'msgid' and 'msgstr' entries do not both end with '\n'
+#
+# This script tries to compile each updated PO file, and revert PO files that cannot be compiled by GNU gettext.
+# So the invalid PO files won't fail Basic Build CI test and block merging the i18n update pull requests.
+
+function discard_po() {
+    echo Discarding $1
+    git restore $1
+}
+
+for i in $(git diff --name-only lang/po/*.po); do
+    msgfmt -o /dev/null $i || discard_po $i
+done
+