Movatterモバイル変換

[0]ホーム
URL:

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

 Sign up
Appearance settings

Add script for handling translations#195

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
hugovk merged 15 commits intopython:mainfromrffontenelle:add-localization
Oct 28, 2024

Conversation

rffontenelle
Copy link
Contributor

@rffontenellerffontenelle commentedAug 1, 2024•
edited
Loading

This is an initial take on the enable localization initiative adding a script to ease the babel call.

Usage:

python babel_runner.py extract# generate translations template file (.pot)python babel_runner.py init -l pt_BR# generate a translation file (.po) for Brazilian Portuguesepython babel_runner.py update# update all .po files using the existing .pot; orpython babel_runner.py update -l pt_BR# update only the pt_BR.popython babel_runner.py compile# compile all .po files using the existing .pot; orpython babel_runner.py compile -l pt_BR# compile only the pt_BR.po

See#194

Copy link
Member

@hugovkhugovk left a comment

Choose a reason for hiding this comment

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

Thanks for this! Please could you add it somehow to the CI, so we know it at least runs without errors?

I was thinking if this script could be converted intoMakefile commands, but then we won't have Windows support, so that's no good. But we could add Makefile commands to call this script. This can be left for later though.

@rffontenelle
Copy link
ContributorAuthor

rffontenelle commentedAug 2, 2024•
edited
Loading

@hugovk workflow job added to tests.yml. My intention is to show the script works, but we can shape it better.

@Wulian233
Copy link

It looks like this PR hasn't had a new activity in a while, is it finished now?

Copy link
Member

@hugovkhugovk left a comment

Choose a reason for hiding this comment

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

A few nits, then I think we can merge this and iterate if needed, unless anyone else has any last comments?

@Wulian233
Copy link

Wulian233 commentedOct 27, 2024•
edited
Loading

In the Simplified Chinese documentation, there is an issue regarding hard-coded (edit: not hard-coded) strings such as "added in version". that cannot be translated. Can this PR solve this problem?

python/python-docs-zh-cn#364

@rffontenelle
Copy link
ContributorAuthor

In the Simplified Chinese documentation, there is an issue regarding hard-coded strings such as "added in version" and "deprecated in version" etc. that cannot be translated. Can this PR solve this problem?

python/python-docs-zh-cn#364

No, that's a Sphinx string and it is not translated to zh_CN:https://github.com/sphinx-doc/sphinx/blob/116a430caca98580c8877cf7b0ba270eae9604bf/sphinx/locale/zh_CN/LC_MESSAGES/sphinx.po#L1951

Consider contributing inhttps://explore.transifex.com/sphinx-doc/sphinx-1/

Wulian233 reacted with thumbs up emoji

rffontenelleand others added4 commitsOctober 28, 2024 08:23
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>Co-authored-by: Ezio Melotti <ezio.melotti@gmail.com>
Otherwise init command will be skipped
As one need to have POT before initializing PO
@rffontenelle
Copy link
ContributorAuthor

A few nits, then I think we can merge this and iterate if needed, unless anyone else has any last comments?

That's all from my side. Just a few minor changes.

hugovk reacted with hooray emoji

Copy link
Member

@hugovkhugovk left a comment

Choose a reason for hiding this comment

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

Thank you!

@Wulian233
Copy link

It looks great! I have a question: where should we translate this theme, and how will the translation automatically sync to this repository?

@rffontenelle
Copy link
ContributorAuthor

@Wulian233 I mentioned that in the issue and it is yet to be defined.

Some projects coordinate translation efforts in the own forge via pull/merge requests — not optimal for the translators. There is also translation platforms like Weblate, Transifex and others.

My vote would be to use Weblate, which provides "libre hosting" for free software and features automatic opening and updating pull request for contributed translations. Tasks involve: setting up Weblate project (I can help with/do it), maintainers here update POT and give time before releasing to receive translations; and merging translation pull requests from Weblate.

@hugovkhugovk merged commit3c53d33 intopython:mainOct 28, 2024
14 checks passed
@rffontenellerffontenelle deleted the add-localization branchOctober 28, 2024 14:24
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@AA-TurnerAA-TurnerAA-Turner left review comments

@ezio-melottiezio-melottiezio-melotti left review comments

@Wulian233Wulian233Wulian233 left review comments

@hugovkhugovkhugovk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@rffontenelle@Wulian233@hugovk@AA-Turner@ezio-melotti
[8]ページ先頭
©2009-2025 Movatter.jp