apptools-lab / AppWorks

🐻 基于 VS Code 插件的前端研发工具集,站点国内镜像:https://apptools.gitee.io
https://appworks.site/
MIT License
996 stars 181 forks source link

@appworks/doctor扫描结果不准确 #992

Closed wyong03112636 closed 2 years ago

wyong03112636 commented 2 years ago

插件名/@appworks/doctor:

您的本地环境信息/

您遇到的问题及复现步骤

扫描结果不准确,分数太极端。除了100就是0 image image

您期待的正确结果

与vscode插件的扫描件结果保持一致 image

wyong03112636 commented 2 years ago

补充: 看了@appworks/doctor 0.4.0包的源码,发现eslint规则最终是使用@applint/eslint-config包提供的规则。并没有提供vue2框架使用的lint规则,希望能给出一套适用vue2的lint规则

luhc228 commented 2 years ago
  1. doctor 插件暂时未更新到最新的 @appworks/doctor npm 依赖,近期会支持一下。
  2. 100 分应该就是你的文件没有 Lint 问题了,maintainability 为0 是因为目前没有扫描代码可维护度,我们考虑把它去掉吧
wyong03112636 commented 2 years ago

感谢回复

luhc228 commented 2 years ago

感谢回复

项目的依赖版本有点旧。比如一些 babel-preset-* 的依赖(里面依赖很多,搞了很久都不清楚哪个引入了哪个了)。

image

你可以看下我这个最小的 demo ,是可以 work 的:https://github.com/luhc228/weex-applint-demo

wyong03112636 commented 2 years ago

总结一下:

image

wyong03112636 commented 2 years ago

发现一个问题: 通过disable的方式屏蔽报错或者警告,插件给出的分数会有所改善。通过doctor脚步跑出来的分数不会收到影响。 image

wyong03112636 commented 2 years ago

image eslint配置里可以加一下weex全局变量吗?

fyangstudio commented 2 years ago

doctor 会合并用户 @Iceworks/spec 的 eslintrc 配置,可以在 eslintrc 中配置下

wyong03112636 commented 2 years ago

image 我的eslint配置是这样的,使用vscode doctor插件检测是没问题的,但是用脚本的方式去检测就会有这个问题

fyangstudio commented 2 years ago

1001

wyong03112636 commented 2 years ago

总结一下:

  • eslint方面,升级了babel的依赖,可以正常给出分数了。
  • maintainability方面,准备去掉这功能。
  • repeatability方面,分数还是不太对。我用你的demo复制了几行重复的代码给出结果还是100分

image

@fyangstudio @luhc228 repeatability结果不准的问题,有时间给解决下