Openlibrary: 让我们用 eslint 来检查我们的 JavaScript！

创建于 2018-10-17 · 5评论 · 资料来源: internetarchive/openlibrary

Stylelint 为我们提供了各种 CSS 问题的保护。让我们为我们的 JS 做同样的事情。
eslint 是最成熟的，它提供了我们认为需要时可以使用的规则列表

npm install eslint --save-dev

在 package.json 中，添加一个新命令

"lint:js": "eslint openlibrary/plugins/openlibrary/js/"

添加一个 .eslintrc.json

{
    "root": true,
    "env": {},
    "globals": {
    },
    "rules": {
        "no-extra-semi": "warn"
    }
}

我们将暂时将错误设置为警告，一旦配置被合并，我们应该与某人协调运行修复和合并以避免任何合并冲突！

Good First Issue Linting JavaScript 1 hacktoberfest

资料来源

jdlrobson

所有5条评论

你好。你能指派我解决这个问题吗？

0x15F9 于 2018-10-18

问题：为什么不为eslint使用预设之一？像https://standardjs.com或https://github.com/airbnb/javascript

tamtamchik 于 2018-10-20

从长远来看，我想，但 javascript 的状态如此糟糕，应用这样的预设会造成很多混乱（代码审查、rebase 冲突）和手动工作（大多数预设无法自动修复）。我认为手动添加规则可以帮助我们以一种有凝聚力和教育性的方式一点一点地攻击它（我们一点一点地了解我们代码中的问题）。

我想我们可以使用 linting 来解决最常见和最紧迫的问题（可能导致代码错误的 lint），然后在闲暇时应用（我们选择的）预设。

这有意义吗？

jdlrobson 于 2018-10-20

👍1

当然，确实如此。代码形状可能非常糟糕。

当这个 PR 被合并时，我可以解决一些 linting 的问题，只需命名 em。
例如，将以no-extra-semi开头；)

tamtamchik 于 2018-10-20

👍1

@tamtamchik我可以稍后创建一个问题，但是在“可能的错误”snd 和自动修复下的任何内容都是公平的游戏。

我们应该与@mekarpeles协调运行和提交自动修复程序，并使 eslint 错误而不是警告。

我要我们摆脱悬而未决的果实！

jdlrobson 于 2018-10-21

🎉1

此页面是否有帮助？

0 / 5 - 0 等级