Skip to content

Commit ba1beeb

Browse files
DanWahlinCopilot
andcommitted
fix(translations): require grade body update
Updates Translation Polisher loop prevention so it can still add the managed Translation Quality Review section when the PR has already been polished but lacks per-file grades. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent c5c67c5 commit ba1beeb

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

.github/workflows/translation-polisher.md

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,14 +72,16 @@ If this workflow is triggered by `pull_request`, inspect the triggering pull req
7272

7373
## Loop prevention
7474

75-
Before editing, inspect the pull request's latest commit and current diff.
75+
Before editing, inspect the pull request's latest commit, current diff, labels, and body.
7676

77-
Stop with a no-op if either of these is true:
77+
Stop with a no-op only if both of these are true:
7878

79-
1. The latest commit appears to be from this Translation Polisher workflow.
80-
2. The translated Markdown already satisfies the quality checklist below and no changes are needed.
79+
1. The translated Markdown already satisfies the quality checklist below and no file changes are needed.
80+
2. The pull request body already contains an up-to-date `## Translation Quality Review` section with one grade row for every changed translated Markdown file.
8181

82-
Do not add churn. If the translation is already good enough, leave it unchanged.
82+
If the latest commit appears to be from this Translation Polisher workflow but the pull request body is missing the `## Translation Quality Review` section, do not edit files. Still review and grade the changed translated Markdown files, update the pull request body, and add a concise comment if useful.
83+
84+
Do not add churn. If the translation is already good enough and the PR body already has current grades, leave it unchanged.
8385

8486
## Files you may change
8587

0 commit comments

Comments
 (0)