zhongdeming428 / Jerry

A lightweight but useful JavaScript library,just for fun!:memo::sparkles:
https://zhongdeming428.github.io/Jerry/
MIT License
9 stars 3 forks source link

BugsList #3

Open zhongdeming428 opened 5 years ago

zhongdeming428 commented 5 years ago
KarthusLorin commented 5 years ago

@zhongdeming428 我发现有部分错别字,需要另拉分支修改吗?

zhongdeming428 commented 5 years ago

@KarthusLorin 如果可以的话,感激不尽:rose:

KarthusLorin commented 5 years ago

@zhongdeming428 isPlainObjectisObjectzhee 这两个函数我感觉有更好的实现方法,可以按照vue源码中的实现方式来实现,我明天有空改改吧~

KarthusLorin commented 5 years ago

@zhongdeming428 我发现很多地方没有遵循 eslint 的规则,个人认为需要定一个 eslint 规则,比如函数括号后要不要加空格之类的,或者直接使用 airbnb 的规则?

zhongdeming428 commented 5 years ago

@KarthusLorin isPlainObject 是参照 redux 方案实现的,不知道 Vue 是怎么实现的,你可以试试看。另外 eslint 之前设置了规则,但是 npm 脚本没设置好,写成了只校验 src/index.js,所以其他模块没校验到:cry:。现在修改了脚本,能够校验了,另外有问题的代码我也修复了。规则不用改,现在能够校验到。