Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Align FR translation to 0.14.0 #1266

Merged
merged 2 commits into from
Jan 31, 2025
Merged

Conversation

cregouby
Copy link
Collaborator

No description provided.

@dfalbel
Copy link
Member

dfalbel commented Jan 31, 2025

Thanks @cregouby ! What's the process for generating those? Should I always do it before release?

@dfalbel dfalbel merged commit a3052da into mlverse:main Jan 31, 2025
7 checks passed
@cregouby
Copy link
Collaborator Author

Hello @dfalbel

Thanks for merging my (late) contribution.

the complete process is in here : https://michaelchirico.github.io/potools/articles/developers.html
the TL;DR process, for update only, is

#|label: developer extracts messages
library(potools)
# update message translation file
po_update(here::here()))

This updates the common (english) po/R-torch.pot file.

Now each translator is ready to work to update its own po/R-__LANG__.po (clear text) translation file, and then to compile it into the target folder with po_compile(here::here())

About the release process, I would add two translation tasks

  •  update the po/R-torch.pot message file
  • inform the translators and collect back updated translations

@cregouby cregouby deleted the translation/fr_0.14.0 branch January 31, 2025 16:22
@dfalbel
Copy link
Member

dfalbel commented Jan 31, 2025

Thanks, I'll make sure to do that for the next release. I'll make a patch release of 0.14 soon too, to include those changes.

@cregouby
Copy link
Collaborator Author

cregouby commented Jan 31, 2025

I have propose a P.R. to usethis::use_release_isssue()... in r-lib/usethis#2099

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.

2 participants