From da58fb3d88f17b9a28b1deb7f4588447f3cf88e4 Mon Sep 17 00:00:00 2001 From: Michael Geers Date: Tue, 14 Oct 2025 10:09:35 +0200 Subject: [PATCH] chore: proper escaping, see #24300, drop actions/ai-inference and use plain js --- .github/workflows/language-reminder.yml | 32 ++++++++++++++++++------- 1 file changed, 23 insertions(+), 9 deletions(-) diff --git a/.github/workflows/language-reminder.yml b/.github/workflows/language-reminder.yml index 601ed0320..b5ed644d2 100644 --- a/.github/workflows/language-reminder.yml +++ b/.github/workflows/language-reminder.yml @@ -91,13 +91,27 @@ jobs: if (analysis.requires_reminder) { const comment = context.payload.comment; - await github.rest.pulls.createReviewComment({ - owner: context.repo.owner, - repo: context.repo.repo, - pull_number: context.payload.pull_request.number, - body: `@${comment.user.login} 🇬🇧 Please use English so everyone can participate. See [Contributing Guidelines](https://github.com/${context.repo.owner}/${context.repo.repo}/blob/master/CONTRIBUTING.md#communication-language).`, - commit_id: comment.commit_id, - path: comment.path, - in_reply_to: comment.id - }); + const reminderMessage = `@${comment.user.login} 🇬🇧 Please use English so everyone can participate. See [Contributing Guidelines](https://github.com/${context.repo.owner}/${context.repo.repo}/blob/master/CONTRIBUTING.md#communication-language).`; + + // Handle different event types + if (context.eventName === 'pull_request_review_comment') { + // Review comment on PR - reply in thread + await github.rest.pulls.createReviewComment({ + owner: context.repo.owner, + repo: context.repo.repo, + pull_number: context.payload.pull_request.number, + body: reminderMessage, + commit_id: comment.commit_id, + path: comment.path, + in_reply_to: comment.id + }); + } else if (context.eventName === 'issue_comment' && context.payload.issue.pull_request) { + // Comment on PR issue - post as issue comment + await github.rest.issues.createComment({ + owner: context.repo.owner, + repo: context.repo.repo, + issue_number: context.payload.issue.number, + body: reminderMessage + }); + } }