From c2bc4008fc9d2787990c1f3526cf771a13756b51 Mon Sep 17 00:00:00 2001
From: Jesse Hallam <jesse.hallam@gmail.com>
Date: Thu, 19 Oct 2023 14:40:06 -0300
Subject: [PATCH] prevent modifying non-English translation files (#25002)

---
 .github/workflows/i18n-ci-pr.yml       | 20 +++++++++++++++++
 .github/workflows/i18n-ci-template.yml | 30 ++++++++++++++++++++++++++
 2 files changed, 50 insertions(+)
 create mode 100644 .github/workflows/i18n-ci-pr.yml
 create mode 100644 .github/workflows/i18n-ci-template.yml

diff --git a/.github/workflows/i18n-ci-pr.yml b/.github/workflows/i18n-ci-pr.yml
new file mode 100644
index 0000000000..d2c003e653
--- /dev/null
+++ b/.github/workflows/i18n-ci-pr.yml
@@ -0,0 +1,20 @@
+name: i18n CI PR
+on:
+  pull_request:
+    paths:
+      - "server/i18n/**"
+      - "webapp/channels/src/i18n/**"
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+# This file just imports the template yml
+# and runs it with concurrency. We have to do this in this way
+# because the concurrency label cannot be added conditionally
+# and it _always_ cancels pending workflows. So master CI builds
+# always kept getting canceled.
+
+jobs:
+  pr-ci:
+    uses: ./.github/workflows/i18n-ci-template.yml
+    secrets: inherit
diff --git a/.github/workflows/i18n-ci-template.yml b/.github/workflows/i18n-ci-template.yml
new file mode 100644
index 0000000000..83ad3d5eab
--- /dev/null
+++ b/.github/workflows/i18n-ci-template.yml
@@ -0,0 +1,30 @@
+# Base CI template which is called from i18n-ci-pr.yml
+
+name: i18n CI Template
+on:
+  workflow_call:
+
+jobs:
+  check-files:
+    name: Check only English translation files changed
+    runs-on: ubuntu-22.04
+    if: github.event.pull_request.user.login != 'weblate' # Allow weblate to modify non-English
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@ac593985615ec2ede58e132d2e21d2b1cbd6127c # v3.3.0
+
+      - name: Get changed files
+        id: changed-files
+        uses: tj-actions/changed-files@95690f9ece77c1740f4a55b7f1de9023ed6b1f87 # v39.2.3
+        with:
+          files: |
+            server/i18n/*.json
+            webapp/channels/src/i18n/*.json
+            !server/i18n/en.json
+            !webapp/channels/src/i18n/en.json
+
+      - name: Check changed files
+        if: steps.changed-files.outputs.any_changed == 'true'
+        run: |
+          echo "::error title=Non-English i18n files changed::Only PRs from weblate should modify non-English translation files."
+          exit 1