From f3942d35084a6c37a5adec3bec41e32c173ec653 Mon Sep 17 00:00:00 2001 From: Ashley Harrison Date: Fri, 16 May 2025 10:08:21 +0100 Subject: [PATCH] maybe? --- .github/workflows/i18n-crowdin-download.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/i18n-crowdin-download.yml b/.github/workflows/i18n-crowdin-download.yml index a3ff06330dd..e437a057027 100644 --- a/.github/workflows/i18n-crowdin-download.yml +++ b/.github/workflows/i18n-crowdin-download.yml @@ -150,7 +150,7 @@ jobs: # - the PR does not modify the en-US locale # TODO make these paths inputs to the workflow when we move this to a shared repo run: | - filesChanged=$(gh pr diff --name-only ${{ env.PR_URL }}) + filesChanged="$(gh pr diff --name-only "${{ env.PR_URL }}")" if [[ $(echo "$filesChanged" | grep -cv -e 'public/locales/[a-zA-Z\-]*/grafana.json' -e 'public/app/plugins/datasource/azuremonitor/locales/[a-zA-Z\-]*/grafana-azure-monitor-datasource.json') -ne 0 ]]; then echo "Non-i18n changes detected, not approving" @@ -163,8 +163,8 @@ jobs: fi echo "Approving and enabling automerge" - gh pr review "${PR_URL}" --approve - gh pr merge --auto --squash "${PR_URL}" + gh pr review "${{ env.PR_URL }}" --approve + gh pr merge --auto --squash "${{ env.PR_URL }}" env: GITHUB_TOKEN: ${{ steps.generate_approver_token.outputs.token }} PR_URL: ${{ steps.crowdin-download.outputs.pull_request_url }}