已合并
[pre-commit] 增强静态检查 #375
jason lyu创建于 7月1日
[pre-commit] 增强静态检查 #375
已合并
Pull Request已成功合入, 合并人@wangyang
（感谢 jason lyu 的贡献）7月1日 添加了label：ascend-cla/yes
ascend-robot
7月1日 评论：
7月1日 评论：
Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-Ascend/MindIE-PyMotor | ✅ ganglv, hu-xinyi_555 (2/2) | ✅ ganglv (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
codeDogPro, thanks for your pull request. All authors of the commits have signed the CLA. 👍
7月1日 修改了pull request 的描述
7月1日 修改了pull request 的描述
ascend-robot
7月1日 评论：
7月1日 评论：
ascend docs pipeline is running...
此处折叠了122条消息 查看更多
7月8日 添加了label：approved
hu-xinyi_555
7月8日 评论：
7月8日 评论：
/lgtm
7月8日 添加了label：lgtm
7月8日 合入了pull request
1. 合入背景
1、增加license header格式检查【自动修复】
2、增加现代python语法糖检查【自动修复】
3、增加docs文档格式检查
2. 修改内容
注意！两个检查的python脚本必须加shebang，这是pre-commit的要求
3. 资料变更
4. 接口变更
5. 测试结果
1、header：
达成自动修复
2、model typing：
3、docs：
文档检查不会自动修复，仅告知问题所在
6. CheckList
[ ] 代码注释完备
[ ] 正确记录维测日志
[ ] 是否有UT用例
[ ] 若涉及多线程场景，考虑了并发场景，不存在死锁问题