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

Translate errorhandling #46

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

qiufengyu
Copy link
Contributor

@qiufengyu qiufengyu commented May 30, 2021

  • Poedit 提示有问题:msgid 和 msgstr 中的指定格式数量不匹配,删除 #, python-format 一行后该错误会消失,PR 中暂时仍然保留。make html 无错误提示。

    #: ../../errorhandling.rst:4
    #, python-format

  • Error handler 我暂时翻译为错误处理程序(Google 翻译的结果),将根据 Translate blueprints - frostming #41 的结果调整,也欢迎探讨!
  • 有的地方参考了 1.x 的翻译版本。
  • 在 README.md 中勾选翻译的章节条目(把方括号里的空格替换为 x).
  • 在本地生成文档并预览输出,处理所有错误和异常。
  • 运行 pre-commit 钩子,处理所有问题。
  • 更新 .po 文件顶部的 Last-Translator 字段。

docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
docs/locales/zh_CN/LC_MESSAGES/errorhandling.po Outdated Show resolved Hide resolved
@greyli
Copy link
Member

greyli commented May 31, 2021

Poedit 提示有问题:msgid 和 msgstr 中的指定格式数量不匹配,删除 #, python-format 一行后该错误会消失

我没用 Poedit,不太明白什么意思,方面详细说说吗?

@greyli
Copy link
Member

greyli commented May 31, 2021

试了一下,原来英文原文是 Poedit 变动的(#50)。

@qiufengyu
Copy link
Contributor Author

关于 PoEdit 引起的英文原文的变动都回滚了。几处翻译建议也都根据大佬的意见修改过了

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.

None yet

2 participants