From 7f99cbc3d6acf8c4cd7a6c4217fe267d719d4401 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 17 Jul 2026 23:14:36 +0000 Subject: [PATCH] fix(ci): include commons translations in the Crowdin PR The Crowdin action now downloads the commons module's Compose-resource translations too, but the create-pull-request step's add-paths still only covered amethyst/ and docs/. New untracked commons files created by the Crowdin action (run as root in its container) were therefore left out of the commit and shelved by git stash --include-untracked, which failed with 'Permission denied' when trying to remove the root-owned files. Add commons/src/commonMain/composeResources/**/strings.xml to add-paths so those translations are committed into the PR instead of being stashed. --- .github/workflows/crowdin.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/crowdin.yml b/.github/workflows/crowdin.yml index f225b047fc..269ba18404 100644 --- a/.github/workflows/crowdin.yml +++ b/.github/workflows/crowdin.yml @@ -61,6 +61,7 @@ jobs: branch: l10n_crowdin_translations add-paths: | amethyst/src/main/res/**/strings.xml + commons/src/commonMain/composeResources/**/strings.xml docs/changelog/translators.json commit-message: 'chore: sync Crowdin translations and seed translator npub placeholders' title: 'New Crowdin Translations'