-
Notifications
You must be signed in to change notification settings - Fork 837
update ci_manual #5125
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
update ci_manual #5125
Conversation
感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-5125.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html |
|
||
### MAC | ||
|
||
#### PR-CI-Mac-Python3 |
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.
好像多删了很多条目?再核对一下~
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.
子标题也都没了,比如 MAC 平台测试项、华为 NPU 测试项等
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.
已经修复
git commit -m 'test=document_fix' | ||
``` | ||
- 测试项后出现绿色的对勾,表示本条测试项通过。 | ||
- 测试项后出现红色的叉号,并且后面显示 `Required`,则表示本条测试项不通过(不显示 `Required` 的任务未通过,也不影响代码合入,可不处理)。 | ||
|
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.
这里补充一个说明:
注意:
PR-CI-APPROVAL
和PR-CI-Static-Check
这两个 CI 测试项可能需要飞桨相关开发者 approve 才能通过,除此之外请确保其他每一项都通过,如果没有通过,请通过报错信息自查代码。
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.
已经添加
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.
ok
更新CI使用文档