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

Update to Python 3.12 #301

Merged
merged 7 commits into from
Aug 3, 2023
Merged

Update to Python 3.12 #301

merged 7 commits into from
Aug 3, 2023

Conversation

BlueGlassBlock
Copy link
Collaborator

Resolves #289

@BlueGlassBlock
Copy link
Collaborator Author

BlueGlassBlock commented Feb 27, 2023

这个 PR 用 transifex/cli 替代了已经停止维护的 transifex-client

transifex/cli.tx/config 文件有较大改动,可参考我 fork 的各个分支。

@zhsj
Copy link
Collaborator

zhsj commented Feb 27, 2023

官方客户端的问题是非常非常慢,很难在一小时内同步完所有版本。

现在新版本有改善了吗?(好像拿Go重新了,可能效率没问题了)。

你可以在你自己的repo里跑一下,贴一下结果吗?

@BlueGlassBlock
Copy link
Collaborator Author

tx pull -f 对于一个版本只需要几分钟,而且这是默认的 5 并发数。

我发现我重构的配置迁移脚本有些 bug,先 draft 修几天

@ProgramRipper
Copy link

ProgramRipper commented Feb 28, 2023

根据 sphinx-lint 的结果,当下文档里有 1346 个 reStructuredText 语法问题。我准备用脚本批量解决一些特征明显的问题。

@ProgramRipper
Copy link

ProgramRipper commented Mar 1, 2023

根据 sphinx-lint 的结果,当下文档里有 1346 个 reStructuredText 语法问题。我准备用脚本批量解决一些特征明显的问题。

我已经将更改上传到 transifex,我已经可以在本地手动构建文档了。

在修复过程中,我发现了大量机翻、明显不懂 reStructuredText 语法的翻译和不了解技术细节的错误翻译。

我认为文档翻译亟待复核。

@BlueGlassBlock BlueGlassBlock marked this pull request as ready for review March 1, 2023 09:22
@BlueGlassBlock
Copy link
Collaborator Author

经过 @ProgramRipper 修复的 3.7 与 3.11 文档已经可以在我的 fork 构建,我认为这个 PR 可以合并了

@BlueGlassBlock
Copy link
Collaborator Author

所有版本均可正常构建了

cc @zhsj

@silkriver
Copy link

我看到现在311正常,其他的有报错

@BlueGlassBlock
Copy link
Collaborator Author

https://github.com/BlueGlassBlock/python-docs-zh-cn/actions/runs/4657587611

应该是18小时前有人提交了包含语法错误的翻译导致之后的构建错误

@BlueGlassBlock
Copy link
Collaborator Author

查明并非翻译文本的问题,而是构建出现了错误。

我会检查依赖等其他因素进行排查。

@BlueGlassBlock
Copy link
Collaborator Author

我看到现在311正常,其他的有报错

已经定位到有问题的依赖

pygments/pygments#2407

@Jiuh-star
Copy link

Jiuh-star commented May 13, 2023

已经定位到有问题的依赖

pygments/pygments#2407

Pygments 2.15.1 似乎解决了该问题,请问有最新的进展吗?

@BlueGlassBlock
Copy link
Collaborator Author

Pygments 2.15.1 似乎解决了该问题,请问有最新的进展吗?

CI 页 表明除了 Python 3.10 以外的 CI 构建均已恢复正常,3.10 的翻译经过修正后应该就可以正常构建了。

@Jiuh-star
Copy link

Jiuh-star commented May 13, 2023

官方客户端的问题是非常非常慢,很难在一小时内同步完所有版本。

现在新版本有改善了吗?(好像拿Go重新了,可能效率没问题了)。

你可以在你自己的repo里跑一下,贴一下结果吗?

Transifex CLI 用 Go 重构之后速度没有明显改善,平均也需要 1~2 分钟左右的时间,花费的总时间上与旧版本相当,但使用新版本确实使得代码更好维护了 ( 虽然能跑就行... )。

目前一个 Workflow 平均需要 8~15 分钟左右,时间大部分花费在 Sphinx 的构建上,更换 Transifex 客户端无伤大雅。

Workflows

The output of the update.sh in a workflow

依赖问题已经解决,我认为这个 PR 可以合并。

Copy link

@Jiuh-star Jiuh-star left a comment

Choose a reason for hiding this comment

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

README.rst 中可能存在的 icon 显示问题

README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
@rffontenelle
Copy link
Contributor

rffontenelle commented May 28, 2023

3.12 now available in default project (python-newest). Consider branching 3.11, but also enabling 3.12 as default version.

@BlueGlassBlock BlueGlassBlock changed the title Update to Python 3.11 Update to Python 3.12 May 29, 2023
@BlueGlassBlock
Copy link
Collaborator Author

3.12 now available in default project (python-newest). Consider branching 3.11, but also enabling 3.12 as default version.

Thanks for elaborating that, I have added corresponding workflow file. I'll sync 3.11 configuration file with python-311 on my branch when translation migration completes.

@BlueGlassBlock
Copy link
Collaborator Author

在我的 repo 更新 3.11 了分支以使用 transifex 的 python-311 项目,CI 全部通过,我认为这个 PR 已经完备了。

再次 ping @zhsj 请求 review

不过在 merge 这个 PR 前,需要从我的 repo 同步 3.113.12 两个分支到这里。

@BlueGlassBlock
Copy link
Collaborator Author

在 Python 3.7 EOL 后 transifex 上的 python-37 翻译已存档,停止接收翻译贡献。但因为 repo 的 CI 已经长时间停止运作,我们仍需最后拉取更改后再删除对应的 CI 配置。

同时,我注意到翻译项目的协调员很长时间没有对这个 PR 进行跟进,鉴于 Python 3.12 已经进入 beta 阶段,在此再次 ping 一下 @zhsj

@rffontenelle
Copy link
Contributor

Maybe the current maintainer is unavailable and a new one is needed? If so, consider bringing this up in the in the translation mailing list.

@Dumeng Dumeng merged commit df8186d into python:master Aug 3, 2023
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.

Python 3.11 发布了
8 participants