[CodeStyle] `black -> ruff format` migration - part 23 by gouzil · Pull Request #74704 · PaddlePaddle/Paddle
Navigation Menu
{{ message }}
PaddlePaddle / Paddle Public
- Notifications You must be signed in to change notification settings
- Fork 6k
Merged
SigureMo merged 2 commits intoPaddlePaddle:developfrom
Aug 19, 2025Merged
[CodeStyle] black -> ruff format migration - part 23#74704
SigureMo merged 2 commits intoPaddlePaddle:developfrom
[CodeStyle] black -> ruff format migration - part 23#74704
SigureMo merged 2 commits intoPaddlePaddle:developfrom
Conversation
Copy link
Member
gouzil
commented
Aug 18, 2025
gouzil
commented
PR Category
User Experience
PR Types
Not User Facing
Description
使用性能更好，格式化效果更佳的 ruff format 替代 black 作为新的 formatter，由于我们很早就已经集成了 ruff lint，因此只是减少依赖而不会增加依赖
gouzil
requested a review
from SigureMo
as a code owner
Copy link
paddle-bot
bot
commented
Aug 18, 2025
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.
SigureMo previously approved these changes Aug 18, 2025
SigureMo
dismissed
their stale review
via
b5c4d69
Copy link
Member
SigureMo
commented
Aug 18, 2025
SigureMo commented
Aug 18, 2025恢复 python/paddle/_paddle_docs.py 的修改，因为 #74700 已经删掉相关逻辑，不然的话相互冲突这几个 PR 一个也合不进去
SigureMo approved these changes Aug 18, 2025
Copy link
Member
SigureMo
commented
Aug 18, 2025
SigureMo commented
Aug 18, 2025另外过渡周期不宜太长，不然出现类似的新增文件出现在两边没有 exclude 的情况的话，就会无法 commit，是有风险的
SigureMo
deleted the
ruff/fmt/part-23
branch
Luckycheng222 pushed a commit to Luckycheng222/Paddle that referenced this pull request
Aug 25, 2025…74704) --------- Co-authored-by: SigureMo <sigure.qaq@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 participants