🐛 fix(i18n): fork l10n_crowdin from the resolved integration branch

[scttbnsn]

The Crowdin workflow checks out the triggering ref, which is main whenever a dev→main sync merge touches locale files. The crowdin action then forked l10n_crowdin from main's squash history while opening the PR against the dev base, leaving the translation PR permanently CONFLICTING (#601, #618 — closing #601 didn't cure it because the next run recreated the branch the same broken way).

Fix: after resolving the integration branch, fetch it and switch to it before the crowdin action runs, so l10n_crowdin forks from the branch the PR actually targets. BASE is passed via env (no inline template interpolation in run:).

Covered by a new workflow test asserting the step's position (after base, before the crowdin action), env wiring, and fetch-before-checkout ordering.

Changelog

• 🔧 Changed Crowdin workflow to fetch and check out the resolved integration branch before running the Crowdin action.
• 🐛 Fixed translation PRs being based on the triggering ref during dev → main syncs, preventing persistent conflicts.
• ✨ Added workflow tests covering step ordering, BASE configuration, and fetch-before-checkout behavior.

Concerns

• Verify the resolved BASE branch always exists on origin before checkout.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
drydock-website	Ready	Preview, Comment	Jul 27, 2026 10:32pm
drydockdemo-website	Ready	Preview, Comment	Jul 27, 2026 10:32pm

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 0ac57225-13f9-41a5-882c-3d9dbfb4a681

📥 Commits

Reviewing files that changed from the base of the PR and between 52b73c2 and 46f1642.

📒 Files selected for processing (2)

• .github/tests/crowdin-workflow.test.ts
• .github/workflows/i18n-crowdin.yml

---

📝 Walkthrough

Walkthrough

The Crowdin sync workflow adds a step that fetches the branch resolved by steps.base.outputs.name and checks it out locally before running the Crowdin action. Tests now verify the step ordering, BASE environment value, and ordered git fetch and git checkout -B commands.

Possibly related PRs

• CodesWhat/drydock#548: Refactors loading and lookup of the Crowdin workflow tested here.
• CodesWhat/drydock#592: Changes Crowdin workflow targeting and integration-branch checkout behavior.
• CodesWhat/drydock#599: Updates Crowdin base and integration-branch resolution logic and related tests.

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/v1.6-crowdin-l10n-base

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}