Skip to content

Conversation

@SigureMo
Copy link
Member

  • 更新本地 hooks 版本,对齐主 repo
  • 更新文档中主 repo hooks 版本
  • 不再提及 pre-commit 2.17.0 版本,一直用最新版本也没问题,同时提及替代品 prek,建议使用 uv 的开发者直接使用 prek

Copilot AI review requested due to automatic review settings January 18, 2026 10:20
@paddle-bot
Copy link

paddle-bot bot commented Jan 18, 2026

感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-7681.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明,请参考:飞桨文档预览工具

@github-actions
Copy link

📚 本次 PR 文档预览链接(点击展开)
ℹ️ 预览提醒
请等待 Docs-NEW 流水线运行完成后再点击预览链接,否则可能会看到旧版本内容或遇到链接无法访问的情况。
  • docs/api/paddle/distributed/fleet/Fleet_cn.rst: 点击预览
  • docs/design/others/simple_op_design.md: 点击预览
  • docs/dev_guides/code_contributing_path_cn.md: 点击预览
  • docs/dev_guides/docs_contributing_guides_cn.md: 点击预览
  • docs/dev_guides/git_guides/codestyle_check_guide_cn.md: 点击预览

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the pre-commit hooks configuration and documentation to align with the main PaddlePaddle repository, while modernizing the installation instructions and fixing several typos.

Changes:

  • Updated .pre-commit-config.yaml to bump typos from v1.40.0 to v1.42.0 and ruff from v0.13.0 to v0.14.4
  • Removed version pinning for pre-commit installation (from 2.17.0 to latest) across all documentation files
  • Added recommendation for prek as an alternative to pre-commit for uv users
  • Fixed spelling errors: "Leval" → "Level" and "sholud" → "should"
  • Updated version table in documentation to reflect current tool versions
  • Refined FAQ section with improved formatting and corrected tool references

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.pre-commit-config.yaml Updated typos and ruff hook versions to align with main repo
docs/dev_guides/git_guides/codestyle_check_guide_cn.md Removed pre-commit version pinning, added prek recommendation, updated version table, and improved FAQ formatting
docs/dev_guides/docs_contributing_guides_cn.md Removed pre-commit version pinning from installation instructions
docs/dev_guides/code_contributing_path_cn.md Removed pre-commit version pinning and improved punctuation in Chinese text
docs/design/others/simple_op_design.md Fixed typo "Leval" to "Level"
docs/api/paddle/distributed/fleet/Fleet_cn.rst Fixed typo "sholud" to "should"

Copy link
Collaborator

@ooooo-create ooooo-create left a comment

Choose a reason for hiding this comment

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

LGTM

@SigureMo SigureMo merged commit f47cd2f into PaddlePaddle:develop Jan 18, 2026
11 checks passed
@SigureMo SigureMo deleted the docs/update-pre-commit-hooks-version-and-remove-pre-commit-version-pinning branch January 18, 2026 13:21
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