Less.js: [维护任务] - 检查 ESLint 设置并修复 linting 问题

创建于 2018-06-21 · 6评论 · 资料来源: less/less.js

不久前，我将项目从 JSHint 切换到 ESLint（后者更好维护并提供更多功能）。但是，我将许多设置设为警告，因为我没有时间解决潜在问题。我也没有时间为使代码更易于维护的设置调整 linting。

因此，任务本质上是查看 ESLint 警告，并查看可以安全解决的问题。（例如，许多==相等警告可能无法“修复”到===而不对通过该检查的每个值进行测试。）还为以下内容设置警告会很有帮助将来/正在进行中要解决的问题，例如强制执行 JSDoc 文档。

good first issue medium priority up-for-grabs

资料来源

matthew-dean

所有6条评论

你好，我想抢这个！

umuur 于 2021-01-18

@umuur 加油！

matthew-dean 于 2021-01-19

👍1

@matthew-dean 我们要保留当前的 eslint 配置吗？
我不确定是否需要使用 TypeScript 作为解析器和插件。

umuur 于 2021-01-19

@umuur

我们要保留当前的 eslint 配置吗？

这取决于。如果您想进行不更改代码格式的更改，那很好。但是就警告而言，IMO 这些都是有效的警告，我们要么想解决它们，要么留下关于禁用 eslint 的代码注释以及（最重要的）为什么它在那里无效。

我们要保留当前的 eslint 配置吗？

尽管代码库还没有使用 TypeScript，但根据我的经验，它在转译方面比 Babel 做得更好。就 ESLint 而言...... 我可能这样做是为了将代码库转换为 TS，但现在我不知道是否/何时会发生这种情况。

matthew-dean 于 2021-01-20

👍1

@umuur

顺便说一句，任何可以在参数上添加具有适当类型的 JSDoc 注释的地方，请添加它们！

matthew-dean 于 2021-01-20

@matthew-dean 感谢您的详细评论！会通知您有关更新的信息。

umuur 于 2021-01-20

👍1

此页面是否有帮助？

0 / 5 - 0 等级