Skip to content
This repository was archived by the owner on Oct 7, 2023. It is now read-only.

Conversation

Aarebecca
Copy link
Contributor

支持统一管理update操作

yangtao.yangtao and others added 30 commits June 24, 2021 17:43
refactor(slider): rename type Image to image
@pearmini
Copy link
Member

话说你不是用自己 github 的账号去提交代码吗?🤔

@Aarebecca
Copy link
Contributor Author

话说你不是用自己 github 的账号去提交代码吗?🤔

纳尼😳

@pearmini
Copy link
Member

话说你不是用自己 github 的账号去提交代码吗?🤔

纳尼😳

image

这应该是你公司的账号吧?😆

@hustcc
Copy link
Member

hustcc commented Jul 14, 2021

@Aarebecca 几个问题:

  1. 很多文件的修改,可以想办法拆分成多个 pr。比这个 pr 可以改成做一个组件的 update 做试点,cr 通过之后,后面的组件改法一个模式,cr 成本会底很多。尽量减少一次大量代码的修改
  2. update 和 attrs 的功能是否重复
  3. clear 和 destroy 的区别?clear 之后的实例还能使用吗?
  4. 单元测试又要合理完备的断言,不是单侧的意义就不大了
  5. 目前每个组件的 update 逻辑有一些区别,是否合理?(有些会 clear 在创建,有些是直接 update)
  6. pr 的标题和描述内容,要贴合实际的内容

@visiky visiky merged commit e309f6b into antvis:master Jul 14, 2021
@Aarebecca Aarebecca deleted the refactor-slider branch July 15, 2021 11:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants