Skip to content

🌐 [translation-sync] TEST: Section removed (06 - minimal) - #381

Closed
mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-10-09T04-36-24-pr-982
Closed

mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-10-09T04-36-24-pr-982

Conversation

@mmcky

@mmcky mmcky commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

This PR contains automated translations from QuantEcon/test-translation-sync.

Source PR

#982 - TEST: Section removed (06 - minimal)

Files Added

  • ✅ .translate/state/lecture-minimal.md.yml

Files Updated

  • ✏️ lecture-minimal.md

⚠️ Target-Only Sections Removed

The following sections exist in the current translation but have no counterpart in the source document, so this sync removes them (the translation mirrors the source's structure). If the source deleted these sections, this removal is correct — merge as usual. If they are human-authored additions you want to keep, move them into a target-only file before merging (see adding content to a translated edition).

  • lecture-minimal.md: مدل‌های اقتصادی

Details

  • Source Language: en
  • Target Language: fa
  • Model: claude-sonnet-5

This PR was created automatically by the translation action.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 04:36
@mmcky mmcky added action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation labels Oct 9, 2026
@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-10-09
Routing: auto-merge — no gating findings; floors met


📝 Translation Quality

Criterion Score
Accuracy 10/10
Fluency 10/10
Terminology 10/10
Formatting 10/10
Overall 10/10

Summary: The modified sections (frontmatter/preamble and the deletion of the Economic Models section) are translated accurately, fluently, and with correct terminology. The YAML translation metadata is correctly added per expected conventions, and the deletion is properly reflected with no leftover content. No issues found in the changed sections. Accurate and complete translation of the preamble content and frontmatter metadata Correct removal of the 'Economic Models' section matching the deletion in the source Appropriate use of formal academic Persian register Proper preservation of YAML frontmatter structure with correctly added translation metadata


🔍 Diff Quality

Check Status
Scope Correct ✅
Position Correct ✅
Structure Preserved ✅
Heading-map Correct ✅
Overall 10/10

Summary: The 'Economic Models' section and its corresponding heading-map entry were correctly removed from the target document, matching the source deletion.


This review was generated automatically by action-translation review mode.

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.

🟢 Approval recommended

The heading and body removals are consistent with each other, and the new state file's section-count (1), source-sha, and synced-at date all match the remaining content and PR metadata.

0 open findings

What changed in this PR

This is an automated translation-sync PR that mirrors a source-document structural change into the Farsi (fa) edition. The source document (PR #982) removed the "Economic Models" section, so this sync removes the corresponding target-only section from the translation and records the sync state. It keeps the translated edition structurally aligned with its English source.

Changes:

  • Removes the Economic Models: مدل‌های اقتصادی heading mapping from the lecture-minimal.md front matter and deletes the corresponding ## مدل‌های اقتصادی body section.
  • Adds .translate/state/lecture-minimal.md.yml recording the sync metadata (mode: UPDATE, section-count: 1).
File Description
lecture-minimal.md Removes the "Economic Models" heading entry and its body section to match the source structure.
.translate/​state/​lecture-minimal.md.yml New per-file sync state; section-count: 1 matches the single remaining heading and source-sha matches the PR metadata.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mmcky

mmcky commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

Closing for test reset

@mmcky mmcky closed this Oct 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants