search

antvis / data-set

state driven all in one data process for data visualization.
MIT License
248 stars 46 forks source link

feat: dataset减轻包改动初稿 #99

Closed arcsin1 closed 3 years ago

arcsin1 commented 3 years ago

实验性改动方式一（待讨论）

风险

需要 @hustcc 帮忙看一下方案一这样是否可行

arcsin1 commented 3 years ago

@hustcc 这种方案看是否可行。其实注册这一步我还有种想法，是外部用到了某个方法自己注册

hustcc commented 3 years ago

逻辑是 ok 的，几个建议：

  1. 所有文件中的方法都使用 export，不使用 export default，保持统一。
  2. connector/index.ts 中只负责 export 所有的 function，也不负责 register，可以新开一个 register 目录，里面包含 connector、transform 等，专门负责注册。
  3. 现有单元测试不变，然后增加纯函数的单元测试（这个工作量很大，原单测也可以覆盖到这些方法，可以暂缓）