KEMBAR78
[CodeStyle] `black -> ruff format` migration - part 15 by gouzil · Pull Request #74669 · PaddlePaddle/Paddle · GitHub
Skip to content

Conversation

@gouzil
Copy link
Member

@gouzil gouzil commented Aug 17, 2025

PR Category

User Experience

PR Types

Not User Facing

Description

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

@gouzil gouzil requested a review from SigureMo August 17, 2025 03:03
@paddle-bot
Copy link

paddle-bot bot commented Aug 17, 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 17, 2025
assert self.pp_degree == 2, (
"For manually set pipeline, only " "pp_degree = 2 is supported."
)
assert (
Copy link
Member

Choose a reason for hiding this comment

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

这里,再使用 ruff 的话还会再变回去,或者说 ruff format 的风格就是

assert xxx, (
    "xxxxxxxx"
)

而不是

assert (
    xxx
), "xxxxxxxx"

Copy link
Contributor

@luotao1 luotao1 left a comment

Choose a reason for hiding this comment

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

川师傅,PFCC群里发的太乙比赛 https://www.taiyi.top/competition-home/wuzhen 可以注册下,领双重奖励。请参考 https://www.taiyi.top/knowledgedetail?id=439&type=2 指引,关联如果碰到问题,可以加技术群。

这个比赛的津贴是实时变化的,具体规则可以见 https://www.taiyi.top/knowledge-content?id=2&docId=452&type=2https://www.taiyi.top/knowledge-content?id=2&docId=450&type=2

⚠️ PR合入飞桨后,需要在太乙界面上提交,才能获得津贴。可以常常刷新提交下。

@SigureMo SigureMo merged commit 8fff414 into PaddlePaddle:develop Aug 17, 2025
119 of 124 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants