Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2.8.0版本eslint异常 #109

Open
programmer-yang opened this issue Sep 22, 2021 · 11 comments
Open

2.8.0版本eslint异常 #109

programmer-yang opened this issue Sep 22, 2021 · 11 comments

Comments

@programmer-yang
Copy link

降到2.7.0版本后正常

@topwood
Copy link

topwood commented Oct 12, 2021

+1

@maqinghao
Copy link

maqinghao commented Oct 19, 2021

使用2.8版本以后 发现 === 写成 == 也没有报错了, 块注释的空格规范也没有了,是去掉了eslint的eslint:recommended继承了吗?很湿疑问? 我看源码 在继承里面没有发现 继承eslint:recommended 的代码,有哪位大神能告知一下吗

@loveyunk
Copy link

好像是 v2.8.0 4fc937c 把包括 eslint-config-airbnb-base 在内的很多 ESLint 配置或插件移除了,导致很多基础 lint 规则失效了

不知道为什么这样? @chenshuai2144

@chenshuai2144
Copy link
Collaborator

为了减少依赖。。。

有什么必要规则缺失了吗

@loveyunk
Copy link

为了减少依赖。。。

有什么必要规则缺失了吗

Airbnb rules 全都没有了啊,这还不叫缺失吗?

@chenshuai2144
Copy link
Collaborator

他也是个规则合集 里面的规则和我们依赖的重复了

@topwood
Copy link

topwood commented Nov 18, 2021

不理解,我已经固定写死2.7.0了

@loveyunk
Copy link

loveyunk commented Nov 18, 2021

他也是个规则合集 里面的规则和我们依赖的重复了

不知道你说的重复是指跟哪个重复了?

现在问题是你把大部分规则都移除了。就像 @maqinghao 上面提到的一些常用的规则都没有了,直接影响到使用者了,你觉得这样是没有问题的吗?

就算你觉得这些规则不是必要的,ESLint 运行慢之类的原因,那这也应该是一个 breaking change 吧

@chenshuai2144
Copy link
Collaborator

chenshuai2144 commented Nov 18, 2021

可以列举一下规则,我看看有没有必要加回来。

现有的lint规则依赖复杂,再加上脆弱的eslint插件机制,导致我们内部一直出问题。所以我精简了一下。

https://www.yuque.com/chenshuai/web/qb2sa6

这里是我的整理

@maqinghao
Copy link

可以列举一下规则,我看看有没有必要加回来。

现有的lint规则依赖复杂,再加上脆弱的eslint插件机制,导致我们内部一直出问题。所以我精简了一下。

https://www.yuque.com/chenshuai/web/qb2sa6

这里是我的整理

对比了一下2.5 与2.12使用的插件,继承规则,个人还是建议 添加上[eslint-config-airbnb-base]这个插件

@starsoul666
Copy link

2.7.0以后的版本extends删除了eslint-config-airbnb-base包,删除这个包是基于什么的考虑

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants