Skip to content

Split AI translation workflow into two commits for easier review#1361

Merged
amilcarlucas merged 3 commits intomasterfrom
improve_AI_translations_review
Mar 7, 2026
Merged

Split AI translation workflow into two commits for easier review#1361
amilcarlucas merged 3 commits intomasterfrom
improve_AI_translations_review

Conversation

@amilcarlucas
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 7, 2026 15:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR splits the AI translation workflow's single "stage all changes" commit into two distinct commits: one for the raw .pot/.po string merges (un-translated), and a second for AI-translated .po updates plus compiled .mo files. This makes the resulting automated PR easier to review, as reviewers can separately inspect the mechanical string-merge from the AI-generated translations.

Changes:

  • Adds a new "Configure git for commits" step and a "Commit .pot and .po string merges" step that creates the first commit (raw merged strings) before AI translations are applied.
  • Removes .pot staging from the old "Stage all changes" step (now renamed "Stage AI translations and compiled .mo files") so only AI-translated .po and .mo files go into the second commit.
  • Updates the peter-evans/create-pull-request commit-message to accurately reflect that the second commit contains AI translations and compiled .mo files.

@amilcarlucas amilcarlucas merged commit e154177 into master Mar 7, 2026
6 of 7 checks passed
@amilcarlucas amilcarlucas deleted the improve_AI_translations_review branch March 7, 2026 16:00
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