Skip to content

Fix crowdin workflow#22519

Open
toy wants to merge 3 commits intodevfrom
fix-crowdin
Open

Fix crowdin workflow#22519
toy wants to merge 3 commits intodevfrom
fix-crowdin

Conversation

@toy
Copy link
Contributor

@toy toy commented Mar 26, 2026

Improvements to crowdin workflow:

@toy toy force-pushed the fix-crowdin branch 5 times, most recently from 7b180d7 to 4531309 Compare March 26, 2026 16:39
@toy toy marked this pull request as ready for review March 26, 2026 19:24
@toy toy requested review from Copilot and oliverguenther March 26, 2026 19:24
@toy toy changed the title Fix crowdin Fix crowdin workflow Mar 26, 2026
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

Updates the Crowdin translation sync workflow to avoid file ownership fixes, normalize downloaded translation YAMLs, and simplify the Portuguese root-key correction script.

Changes:

  • Switch Crowdin GitHub Action to an OPF fork and remove the post-download chown step.
  • Add a workflow step to rewrite downloaded Crowdin YAML files via a Ruby-based rewriter script.
  • Refactor fix_crowdin_pt_language_root_key to centralize GNU/BSD sed -i handling.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
script/i18n/fix_crowdin_pt_language_root_key Refactors sed -i handling into a helper and runs the two substitutions via the helper.
.github/workflows/crowdin.yml Switches to a forked Crowdin action, removes ownership-fix step, and adds a YAML rewrite step after download.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants