Skip to content

i18n: catalogs list each source once again - #2627

Merged
bflatastic merged 1 commit into
mainfrom
fix/i18n-duplicate-rows
Oct 30, 2026
Merged

bflatastic merged 1 commit into
mainfrom
fix/i18n-duplicate-rows

Conversation

@bflatastic

Copy link
Copy Markdown
Member

Main's i18n::ui::tests::catalogs_are_unique_and_keep_the_english_shape fails after batch Q: #2054's own merge of main re-added Simplified/Traditional Chinese rows that main already had.

crates/ui-egui/src/i18n/ui.rs is now the batch-tested catalog: each source once, with #1514's tool names, #2427's Preview labels, #1581's and #2054's rows and #1340's Polish. ("ka", GEORGIAN_PANELS) is back in ROWS; the batch's catalog-merge helper had wrongly folded it into the first "ka" entry. The i18n unit tests pass.

🤖 Generated with Claude Code

…stered

#2054's own merge of main re-added zh-hans/zh-hant rows that main already
had, so catalogs_are_unique_and_keep_the_english_shape failed on main.
The catalog is the batch-tested one, with ('ka', GEORGIAN_PANELS) back in
ROWS (the batch's merge helper had folded it into the first 'ka' entry).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bflatastic
bflatastic merged commit 60f246d into main Oct 30, 2026
3 checks passed
@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@sonarqubecloud

sonarqubecloud Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues
0 New dependency risks

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube Cloud

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