From a036cbb32fcbb33a81535d4367a216867af788ca Mon Sep 17 00:00:00 2001 From: joshhunt Date: Fri, 16 May 2025 10:56:07 +0100 Subject: [PATCH] just trying this up, im pretty sure this is what it was originally --- .github/workflows/i18n-crowdin-download.yml | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/workflows/i18n-crowdin-download.yml b/.github/workflows/i18n-crowdin-download.yml index e437a057027..493b676dac6 100644 --- a/.github/workflows/i18n-crowdin-download.yml +++ b/.github/workflows/i18n-crowdin-download.yml @@ -150,7 +150,10 @@ 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 "$PR_URL")" + + echo "PR URL: $PR_URL" + echo "Files changed: $filesChanged" 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 +166,8 @@ jobs: fi echo "Approving and enabling automerge" - gh pr review "${{ env.PR_URL }}" --approve - gh pr merge --auto --squash "${{ env.PR_URL }}" + gh pr review "$PR_URL" --approve + gh pr merge --auto --squash "$PR_URL" env: GITHUB_TOKEN: ${{ steps.generate_approver_token.outputs.token }} PR_URL: ${{ steps.crowdin-download.outputs.pull_request_url }}