Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
K
kb
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Operations
    • Operations
    • Incidents
  • Analytics
    • Analytics
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • granite
  • kb
  • Wiki
    • Code review
  • ic_info_change

ic_info_change · Changes

Page history
add: 工商变更codereview authored Nov 25, 2021 by 袁波's avatar 袁波
Hide whitespace changes
Inline Side-by-side
Showing with 24 additions and 16 deletions
+24 -16
  • code-review/ic_info_change.md code-review/ic_info_change.md +24 -16
  • No files found.
code-review/ic_info_change.md
View page @ 00b52701
......@@ -5,6 +5,26 @@
3. 规范性
```
### 评审日期
```
2021-11-25
```
### 评审人
```
王鹏举、袁波
```
### 被评审人
```
杨龙斌
```
### 参考链接
```buildoutcfg
请求流程
http://tech.pingansec.com/granite/project-ic-spiders/-/blob/dev_ic_pc_py3/app_ic_pc/doc/zongju_requests.md
```
### 流程
```buildoutcfg
......@@ -38,7 +58,7 @@
b、太多if else 判断就显得代码特别乱, 是否可以抽取出方法处理。
c、detail_url.py parse_response方法, 2个yield,1个return
8. 不足
4. 不足
a、is_need_retry,方法整体上应该有一个返回
......@@ -63,23 +83,11 @@
### 改进落实
```buildoutcfg
缺陷:
规范性:
时间:
负责人:
```
### 已改进问题
```buildoutcfg
缺陷:
规范性:
改进人:
时间:
负责人:
监督人:
```
Clone repository
  • README
  • basic_guidelines
  • basic_guidelines
    • basic_guidelines
    • dev_guide
    • project_build
    • 开发流程
  • best_practice
  • best_practice
    • AlterTable
    • RDS
    • azkaban
    • create_table
    • design
    • elasticsearch
    • elasticsearch
      • ES运维
    • logstash
View All Pages