Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

i18n(sc): AI translation update — sync with messages.pot (2026-06-11)#12906

Merged
mekarpeles merged 1 commit into
masterinternetarchive/openlibrary:masterfrom
i18n/scinternetarchive/openlibrary:i18n/scCopy head branch name to clipboard
Jun 12, 2026
Merged

i18n(sc): AI translation update — sync with messages.pot (2026-06-11)#12906
mekarpeles merged 1 commit into
masterinternetarchive/openlibrary:masterfrom
i18n/scinternetarchive/openlibrary:i18n/scCopy head branch name to clipboard

Conversation

@mekarpeles

Copy link
Copy Markdown
Member

Summary

Syncs the sc (Sardinian) translation file with the current `messages.pot` template and fills in
previously untranslated strings.

AI attribution: All new translations in this PR were generated by `claude-sonnet-4-6`. They should
be reviewed by a native speaker before merging, particularly for tone and idiomatic accuracy.
Format strings (`%(name)s`, `%(count)d`, etc.) and HTML markup were preserved verbatim.

Changes

  • Ran `scripts/i18n-messages update sc` to sync with current `messages.pot`
  • Filled 186 previously untranslated msgid entries
  • Fixed 17 format-string errors and 8 HTML structure errors in fuzzy entries
  • Left fuzzy entries unchanged (marked for human review)

Validation

  • `i18n-messages validate sc` — 0 errors ✓
  • `test_po_files.py -k sc` — 170 passed, 0 failures ✓
  • `pre-commit run --files openlibrary/i18n/sc/messages.po` — clean ✓

Reviewer guidance

Please spot-check translations for:

  1. Accuracy of meaning (does the translation convey the intent?)
  2. Natural phrasing (does it sound like UI text, not literal translation?)
  3. Preserved format strings and HTML

Native speakers: please edit the `.po` file directly on this branch.

🤖 Generated by `claude-sonnet-4-6`

Fills in 186 untranslated strings for sc (Sardinian) using claude-sonnet-4-6.
Synced with current messages.pot via i18n-messages update.
Fixed 17 format-string errors and 8 HTML structure errors in fuzzy entries.

🤖 Translations generated by claude-sonnet-4-6. Please review for accuracy.
All format strings (%(name)s, %(count)d, etc.) preserved verbatim.
HTML markup preserved. Fuzzy entries left for human review.
@mekarpeles
mekarpeles marked this pull request as ready for review June 12, 2026 04:13
Copilot AI review requested due to automatic review settings June 12, 2026 04:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@mekarpeles
mekarpeles merged commit 118990e into master Jun 12, 2026
7 of 8 checks passed
@mekarpeles
mekarpeles deleted the i18n/sc branch June 12, 2026 04:15
@mekarpeles mekarpeles added the Theme: Internationalization (i18n) Making OpenLibrary work for both foreign-language users and books. [managed] label Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Theme: Internationalization (i18n) Making OpenLibrary work for both foreign-language users and books. [managed]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.