Skip to content

[codex] Ensure development translation queue#614

Merged
riderx merged 1 commit into
mainfrom
codex/ensure-dev-translation-queue
May 1, 2026
Merged

[codex] Ensure development translation queue#614
riderx merged 1 commit into
mainfrom
codex/ensure-dev-translation-queue

Conversation

@riderx
Copy link
Copy Markdown
Member

@riderx riderx commented May 1, 2026

Summary

Validation

  • bun run ci:verify:translation
  • bunx prettier --check .github/workflows/deploy-translation.yml

Summary by CodeRabbit

  • Chores
    • Improved deployment infrastructure for translation services by enhancing queue resource management in the GitHub Actions workflow.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: baa4ab8e-7b2b-45b0-a7e4-65f6d63239d2

📥 Commits

Reviewing files that changed from the base of the PR and between c88f4c7 and 74fc4d9.

📒 Files selected for processing (1)
  • .github/workflows/deploy-translation.yml

📝 Walkthrough

Walkthrough

The GitHub Actions deployment workflow for translations is refactored to use a parameterized ensure_queue function instead of inline shell logic. This function conditionally creates Cloudflare queue resources using bunx wrangler queues create and suppresses "already exists" errors. The deployment now provisions both production and development translation queues.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/deploy-translation.yml
Refactored queue creation logic into a reusable ensure_queue function that accepts a queue name parameter. Updated deployment step to create both capgo-translation-refresh and capgo-translation-refresh-development queues with centralized error handling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • [codex] Queue translation cache refreshes #613: Modifies the same deploy workflow step for Cloudflare translation queue provisioning, with the main PR refactoring inline logic into a parameterized function and adding the development queue variant.

Poem

🐰 Two queues now hop where one did play,
A function fine to guide the way,
No more shell scripts cluttering the fold,
Translation refreshes, brave and bold! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title '[codex] Ensure development translation queue' accurately describes the main change—adding queue creation for the development translation queue alongside production.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/ensure-dev-translation-queue

Review rate limit: 0/5 reviews remaining, refill in 58 minutes and 4 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 1, 2026

@riderx riderx marked this pull request as ready for review May 1, 2026 16:44
@riderx riderx merged commit 9d3650f into main May 1, 2026
10 checks passed
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.

1 participant