Skip to content

Add translations for several suggested languages#2918

Closed
jayhesselberth wants to merge 5 commits intomainfrom
more-translations
Closed

Add translations for several suggested languages#2918
jayhesselberth wants to merge 5 commits intomainfrom
more-translations

Conversation

@jayhesselberth
Copy link
Collaborator

Suggestions welcome!

@github-actions
Copy link

github-actions bot commented Oct 4, 2025

@ror-at-ebp

This comment was marked as resolved.

- Update vignette
- Use gender-neutral forms when possible
- Remove geman dialect
- Fix Geman typo
@jayhesselberth
Copy link
Collaborator Author

We could also consider computing the vignette table from the available po files.

@jayhesselberth jayhesselberth requested a review from hadley October 15, 2025 12:50
@hadley
Copy link
Member

hadley commented Oct 15, 2025

@jayhesselberth could you also include the prompt you used? And I'd recommend being explicit in the NEWS bullet about which languages you added, that you used positron assistant, and which model you used.

@hadley
Copy link
Member

hadley commented Oct 15, 2025

I definitely agree that it makes sense to automate the list in the vignette.

@hadley
Copy link
Member

hadley commented Oct 15, 2025

I had a quick go asking claude code to proofread all the existing translations. That also filled in a few missing, found a problem with Turkish (light and dark look to be swapped), and reveals that we need extra instructions about using $1 etc if the order of the words is changed. We might also want some additional advice for languages with gendered nouns.

@hadley
Copy link
Member

hadley commented Oct 15, 2025

First stab at a prompt (for proofreading):

You are an experienced translator. Proofread the translations in po/, making any small fixes immediately and alerting me to any bigger issues. These are GNU gettext translations for pkgdown, a tool used to generate websites for R packages.

Translation advice:

  • For languages that use genedered nouns, do your best to generate inclusive translations, e.g. "Beitragende:r" or "auteur·rice".
  • If you need to change the order of substititions (e.g. %s) from English version, use %1$s to identify the first substituion %2$s, etc. Only use this form when the translated word order is different.

@maelle
Copy link
Collaborator

maelle commented Oct 16, 2025

Thanks! Before merging this, could we please make sure the translations for each language are also reviewed by a native (human 😁) speaker?

@ror-at-ebp
Copy link

The German translation is okay. I noticed two things however (in R-de.po):

  • tutorials is translated with "Tutorien". This is afaik not a common use for that word and usually "Tutorial" or "Anleitung" is used in this context.
  • There is no translation for the msgid "Additional details" (in render.R called author details). A possible translation would be "Zusätzliche Informationen" or "Weitere Details" (to keep it shorter).

@hadley
Copy link
Member

hadley commented Oct 16, 2025

I also agree with @maelle that it would be good to get a human to take a look too. Definitely a lot of potential here for LLMs to speed up the work (and expose some small issues with existing translations) but it would be to good to verify each translation with a human (and if we can't find someone, I think that's a sign that language probably isn't that important for the R community).

Maybe it's worth breaking this up in to a few separate PRs? I'm thinking one to automate the language list in the vignette, another to add a prompt (maybe also in the vignette?) and then one PR for each language?

@jayhesselberth
Copy link
Collaborator Author

Automating the language list in the vignette isn't going to be as easy as I thought, as the only files available to pkgdown (assuming build_site(install = TRUE)) are the binary *.mo files. These can be extracted with msgunfmt but that might be fiddly and not worth adding to pkgdown.

So, if where going to add todo bullets (#2927), I think we should just add a bullet to update the vignette with any language updates.

@jayhesselberth
Copy link
Collaborator Author

Closing in favor of individual PRs.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments