Skip to content

ENH: add review-translations workflow for action-translation - #212

Merged
mmcky merged 4 commits into
mainfrom
feat/review-translations-workflow
Jul 18, 2026
Merged

ENH: add review-translations workflow for action-translation#212
mmcky merged 4 commits into
mainfrom
feat/review-translations-workflow

Conversation

@mmcky

@mmcky mmcky commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the review-translations.yml workflow to enable automatic translation review on PRs created by action-translation.

How it works

  1. The source repo (lecture-python-intro) runs a sync-translations-zh-cn.yml workflow on merged PRs that touch lectures/**/*.md or lectures/_toc.yml. It translates changed content and opens a PR in this repo with the action-translation label.

  2. This reviewer workflow triggers on PRs labeled action-translation, running QuantEcon/action-translation@v0.14.1 in review mode to validate the translation quality.

Setup based on

Required secrets

  • ANTHROPIC_API_KEY — for LLM-based translation review
  • GITHUB_TOKEN — automatically provided

Related

Adds the reviewer workflow that runs on translation PRs
(labeled 'action-translation') opened by the sync workflow
in the source repo (lecture-python-intro).

Uses QuantEcon/action-translation@v0.14.1 in review mode.
Copilot AI review requested due to automatic review settings April 9, 2026 05:35
@netlify

netlify Bot commented Apr 9, 2026

Copy link
Copy Markdown

Deploy Preview for playful-platypus-17d3bb ready!

Name Link
🔨 Latest commit e335722
🔍 Latest deploy log https://app.netlify.com/projects/playful-platypus-17d3bb/deploys/69d7470a31bece0008349946
😎 Deploy Preview https://deploy-preview-212--playful-platypus-17d3bb.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a GitHub Actions workflow to automatically run QuantEcon/action-translation in review mode on incoming translation PRs produced by action-translation, validating zh-cn translations against the lecture-python-intro English source.

Changes:

  • Introduces .github/workflows/review-translations.yml to run translation review on PR events when the PR has the action-translation label.
  • Checks out the PR and runs QuantEcon/action-translation@v0.14.1 with the configured source/target language settings.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/review-translations.yml Outdated
Comment thread .github/workflows/review-translations.yml Outdated
Comment thread .github/workflows/review-translations.yml Outdated
Comment thread .github/workflows/review-translations.yml
@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

@github-actions
github-actions Bot temporarily deployed to pull request April 9, 2026 05:40 Inactive
Address Copilot review feedback:
- Add labeled/reopened to PR trigger types
- Add permissions block (least-privilege)
- Add concurrency group to avoid duplicate LLM calls
- action-translation: v0.14.1 -> v0. The v0 tag floats to the latest
  0.x release (currently v0.17.0), so the review workflow picks up new
  releases without a version bump PR each time. All inputs used here
  (mode, source-repo, source-language, target-language, docs-folder,
  anthropic-api-key, github-token) are unchanged at v0.
- checkout: v4 -> v7, matching the rest of the workflows on main
  after #220.

Verified both refs resolve and the workflow YAML parses.
@netlify

netlify Bot commented Jul 16, 2026

Copy link
Copy Markdown

Deploy Preview for playful-platypus-17d3bb ready!

Name Link
🔨 Latest commit 45af6cf
🔍 Latest deploy log https://app.netlify.com/projects/playful-platypus-17d3bb/deploys/6a5adb3b8d72710008e9b11b
😎 Deploy Preview https://deploy-preview-212--playful-platypus-17d3bb.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
github-actions Bot temporarily deployed to pull request July 16, 2026 06:22 Inactive
Review: job-level concurrency + labeled-event filter (the fleet fix from
lecture-python-programming.fr#7, refined in action-translation v0.17.0
templates) — the previous workflow-level cancel-in-progress could leave a
sync PR with no review when a trailing labeled event cancelled the real run.

Rebase: standard target-repo workflow from action-translation examples/,
required for multi-PR sync waves (see QuantEcon/action-translation#63).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@mmcky
mmcky merged commit e8cb8ed into main Jul 18, 2026
7 checks passed
@mmcky
mmcky deleted the feat/review-translations-workflow branch July 18, 2026 01:48
@github-actions
github-actions Bot temporarily deployed to pull request July 18, 2026 01:52 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants