Skip to content

Conversation

@gouzil
Copy link
Member

@gouzil gouzil commented Aug 18, 2025

PR Category

User Experience

PR Types

Not User Facing

Description

使用性能更好,格式化效果更佳的 ruff format 替代 black 作为新的 formatter,由于我们很早就已经集成了 ruff lint,因此只是减少依赖而不会增加依赖

@gouzil gouzil requested a review from SigureMo as a code owner August 18, 2025 12:10
@paddle-bot
Copy link

paddle-bot bot commented Aug 18, 2025

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Aug 18, 2025
SigureMo
SigureMo previously approved these changes Aug 18, 2025
@SigureMo SigureMo added skip-ci: approval HappyOpenSource 快乐开源活动issue与PR labels Aug 18, 2025
@SigureMo
Copy link
Member

恢复 python/paddle/_paddle_docs.py 的修改,因为 #74700 已经删掉相关逻辑,不然的话相互冲突这几个 PR 一个也合不进去

@SigureMo
Copy link
Member

另外过渡周期不宜太长,不然出现类似的新增文件出现在两边没有 exclude 的情况的话,就会无法 commit,是有风险的

@SigureMo SigureMo merged commit 798a0a3 into PaddlePaddle:develop Aug 19, 2025
119 of 125 checks passed
@SigureMo SigureMo deleted the ruff/fmt/part-23 branch August 19, 2025 08:54
Luckycheng222 pushed a commit to Luckycheng222/Paddle that referenced this pull request Aug 25, 2025
…74704)

---------

Co-authored-by: SigureMo <sigure.qaq@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers HappyOpenSource 快乐开源活动issue与PR skip-ci: approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants