Skip to content

Commit 23c416f

Browse files
mmckyclaude
andcommitted
Correct the label-event comment: one addLabels call, one event per label
Copilot caught this on #10. The comment claimed the sync applies labels 'one call at a time', but pr-creator.ts makes a single addLabels call with an array — it is GitHub that emits one `labeled` event per label, which is what multiplies the runs. Comment only; no behaviour change. The wording came verbatim from the upstream template (connect-existing.md), so the same inaccuracy is there and worth reporting back rather than silently diverging. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent db0cde6 commit 23c416f

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

.github/workflows/review-translations.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,9 @@ on:
1212

1313
jobs:
1414
review:
15-
# Ignore `labeled` events for every other label: a sync applies its labels one call at a
16-
# time, and each one would otherwise start a full (billed) review of the same diff.
15+
# Ignore `labeled` events for every other label: a sync adds its labels in a single
16+
# addLabels call, but GitHub emits one `labeled` event per label, and each would
17+
# otherwise start a full (billed) review of the same diff.
1718
if: >
1819
contains(github.event.pull_request.labels.*.name, 'action-translation') &&
1920
(github.event.action != 'labeled' || github.event.label.name == 'action-translation')

0 commit comments

Comments
 (0)