-
Notifications
You must be signed in to change notification settings - Fork 5.9k
[CodeStyle] black -> ruff format migration - part 15
#74669
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
Conversation
|
你的PR提交成功,感谢你对开源项目的贡献! |
| assert self.pp_degree == 2, ( | ||
| "For manually set pipeline, only " "pp_degree = 2 is supported." | ||
| ) | ||
| assert ( |
There was a problem hiding this comment.
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"There was a problem hiding this 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=2 和 https://www.taiyi.top/knowledge-content?id=2&docId=450&type=2 。
PR Category
User Experience
PR Types
Not User Facing
Description
使用性能更好,格式化效果更佳的 ruff format 替代 black 作为新的 formatter,由于我们很早就已经集成了 ruff lint,因此只是减少依赖而不会增加依赖