-
Notifications
You must be signed in to change notification settings - Fork 5.8k
[Docathon][Fix System Message No.13] #59725
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
[Docathon][Fix System Message No.13] #59725
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
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.
先英文改吧,英文改完中文再同步改
… system_message_10_13
好像 另外,我们要不要像 api_label 一样订一个规范,对于 |
我觉得可以😁😁😁,感觉也不一定,有些是动态图与静态图的区别,不用1 , 2 比较好,尽量包含语义信息 link,要么以api名字开头就行?改的话,中文copy-from也得改,感觉好多😂,不晓得~ |
@SigureMo 一师傅看看呢?要不要对 |
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.
LGTM~总算是清理完啦
PR types
Others
PR changes
Others
Description
link #58237
需要在 #59445 之后合入
@sunzhongkai588